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.

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

Enable custom build output directories with Brownie #366

Merged
merged 3 commits into from
Mar 6, 2023

Conversation

UsmannK
Copy link
Contributor

@UsmannK UsmannK commented Jan 20, 2023

Add support for custom brownie build directories.

crytic-compile fails when it cannot find the build artifacts, but until now did not check the brownie config for custom build directories first.

@CLAassistant
Copy link

CLAassistant commented Jan 20, 2023

CLA assistant check
All committers have signed the CLA.

@UsmannK
Copy link
Contributor Author

UsmannK commented Jan 20, 2023

example brownie config might look like

project_structure:
    build: brownie_out
    contracts: src
    tests: tests/brownie

# use Ganache's forked mainnet mode as the default network
# NOTE: You don't *have* to do this, but it is often helpful for testing
networks:
  default: mainnet-fork

# automatically fetch contract sources from Etherscan
autofetch_sources: True

# path remapping to support imports from GitHub/NPM
compiler:
  solc:
    version: 0.8.14
    optimizer:
      enabled: true

we only care about the build: brownie_out line

@montyly
Copy link
Member

montyly commented Mar 6, 2023

Nice, thanks @UsmannK

@montyly montyly added this pull request to the merge queue Mar 6, 2023
@montyly montyly removed this pull request from the merge queue due to a manual request Mar 6, 2023
@montyly montyly changed the base branch from master to dev March 6, 2023 17:27
@montyly montyly added this pull request to the merge queue Mar 6, 2023
Merged via the queue into crytic:dev with commit 80ed8ad Mar 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants