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

Check whether the libraries are pre-declared in the foundry.toml File #6

Open
andreivladbrg opened this issue Jan 16, 2025 · 0 comments
Open

Check whether the libraries are pre-declared in the foundry.toml File #6

andreivladbrg opened this issue Jan 16, 2025 · 0 comments
Labels
effort: medium Default level of effort. priority: 1 This is important. It should be dealt with shortly. type: feature New feature or request. work: clear Sense-categorize-respond. The relationship between cause and effect is clear.

Comments

@andreivladbrg
Copy link
Member

Since we now have deployable libraries, it would be useful to add a prevention check to verify if the libraries are pre-declared in the foundry.toml file. If they are, we should either comment them out or remove them, depending on which option is easier to implement.
@andreivladbrg andreivladbrg added effort: medium Default level of effort. priority: 1 This is important. It should be dealt with shortly. type: feature New feature or request. work: clear Sense-categorize-respond. The relationship between cause and effect is clear. labels Jan 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
effort: medium Default level of effort. priority: 1 This is important. It should be dealt with shortly. type: feature New feature or request. work: clear Sense-categorize-respond. The relationship between cause and effect is clear.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@andreivladbrg