Skip to content
New issue

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

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

Already on GitHub? # to your account

fix: reference contract name rather than filepath #698

Merged

Conversation

shrugs
Copy link
Contributor

@shrugs shrugs commented Jan 23, 2018

Fixes #625

πŸš€ Description

Removes the filepaths from artifacts.require calls because it's no longer needed.

  • πŸ“˜ I've reviewed the OpenZeppelin Contributor Guidelines
  • βœ… I've added tests where applicable to test my new functionality.
  • πŸ“– I've made sure that my contracts are well-documented.
  • 🎨 I've run the JS/Solidity linters (npm run lint:all:fix) and fixed any issues.

@shrugs shrugs self-assigned this Jan 23, 2018
@shrugs shrugs requested a review from eternauta1337 January 23, 2018 02:56
@shrugs shrugs added the review label Jan 23, 2018
@frangio
Copy link
Contributor

frangio commented Jan 25, 2018

Yes! Thanks @shrugs!

@frangio frangio merged commit f4228f1 into OpenZeppelin:master Jan 25, 2018
@frangio frangio removed the review label Jan 25, 2018
ProphetDaniel pushed a commit to classicdelta/Smart-Contracts that referenced this pull request Mar 9, 2018
@shrugs shrugs deleted the feat/truffle-artifactor-correctly-#625 branch April 10, 2018 18:41
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Use Truffle artifacts.require correctly
2 participants