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

chore: use shared Aragon truffle config #559

Merged
merged 3 commits into from
Oct 29, 2019
Merged

chore: use shared Aragon truffle config #559

merged 3 commits into from
Oct 29, 2019

Conversation

0xGabi
Copy link
Contributor

@0xGabi 0xGabi commented Oct 8, 2019

This truffle config file is still used quite a bit as main configuration and Infura have not been much reliable in some cases reaching the request limits.

Long term we should encourage the use of a truffle specific package once we release it.

@coveralls
Copy link

coveralls commented Oct 8, 2019

Coverage Status

Coverage remained the same at 99.116% when pulling 2af1f0d on aragon-node into 07d309f on next.

@sohkai
Copy link
Contributor

sohkai commented Oct 9, 2019

Given that we will be replacing this Very SoonTM with the separated truffle config, I'd rather just make that change and re-release aragonOS then.

@0xGabi 0xGabi changed the title Use Aragon node Use Aragon truffle file Oct 20, 2019
@0xGabi
Copy link
Contributor Author

0xGabi commented Oct 20, 2019

@sohkai I have updated the PR to use the new truffle v4 package

@sohkai sohkai changed the title Use Aragon truffle file chore: use shared Aragon truffle file Oct 22, 2019
@sohkai sohkai changed the title chore: use shared Aragon truffle file chore: use shared Aragon truffle config Oct 22, 2019
Copy link
Contributor

@sohkai sohkai left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Let's follow this up by replacing the deploy logger :).

package.json Outdated
@@ -50,10 +50,9 @@
"web3-utils": "1.0.0-beta.33"
},
"dependencies": {
"@aragon/truffle-config-v4": "^1.0.0",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's put this in the devDependencies

Copy link
Contributor

@facuspagnuolo facuspagnuolo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@0xGabi 0xGabi merged commit 7042ac7 into next Oct 29, 2019
@welcome
Copy link

welcome bot commented Oct 29, 2019

Congrats on merging your first pull request! Aragon is proud of you 🦅
Eagle gif

@delete-merged-branch delete-merged-branch bot deleted the aragon-node branch October 29, 2019 23:32
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.

4 participants