Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add a --force flag to bypass the compilation cache #2798

Closed
TomAFrench opened this issue Sep 22, 2023 · 0 comments · Fixed by #4054
Closed

Add a --force flag to bypass the compilation cache #2798

TomAFrench opened this issue Sep 22, 2023 · 0 comments · Fixed by #4054
Assignees
@TomAFrench
Labels
enhancement New feature or request nargo Noir's CLI development tool

Comments

@TomAFrench
Copy link
Member

Problem

#2743 adds caching for when a user is recompiling source code for which they already have an artifact for.

Sometimes however we don't want to use this cache and so it would be good to be able to force a recompilation without having to manually delete this artifact.

Happy Case

nargo compile --force should force a full recompilation of the source code even if we already have an artifact.

Alternatives Considered

Some thought should go into handling nargo execute, etc.

Additional Context

No response

Would you like to submit a PR for this Issue?

No

Support Needs

No response
@TomAFrench TomAFrench added the enhancement New feature or request label Sep 22, 2023
@TomAFrench TomAFrench assigned TomAFrench and unassigned jonybur Oct 5, 2023
@Savio-Sou Savio-Sou added nargo Noir's CLI development tool P-LOW labels Oct 31, 2023
@Savio-Sou Savio-Sou removed the P-LOW label Jan 12, 2024
@TomAFrench TomAFrench mentioned this issue Jan 16, 2024
Merged
5 tasks
github-merge-queue bot pushed a commit that referenced this issue Jan 16, 2024
@TomAFrench
feat: add a --force flag to force a full recompile (#4054)
27a8e68 
# Description

## Problem\*

Resolves #2798 

## Summary\*

This PR adds ` --force` flag which will ignore any existing build
artifacts when compiling a circuit.

## Additional Context



## Documentation\*

Check one:
- [x] No documentation needed.
- [ ] Documentation included in this PR.
- [ ] **[Exceptional Case]** Documentation to be submitted in a separate
PR.

# PR Checklist\*

- [x] I have tested the changes locally.
- [x] I have formatted the changes with [Prettier](https://prettier.io/)
and/or `cargo fmt` on default settings.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@TomAFrench TomAFrench

Labels
enhancement New feature or request nargo Noir's CLI development tool
Projects
Noir
Archived in project
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat: add a --force flag to force a full recompile noir-lang/noir
3 participants
@jonybur @TomAFrench @Savio-Sou