Skip to content

fix: reference contract name rather than filepath#698

Merged
frangio merged 3 commits into
OpenZeppelin:masterfrom
shrugs:feat/truffle-artifactor-correctly-#625
Jan 25, 2018
Merged

fix: reference contract name rather than filepath#698
frangio merged 3 commits into
OpenZeppelin:masterfrom
shrugs:feat/truffle-artifactor-correctly-#625

Conversation

@shrugs

@shrugs shrugs commented Jan 23, 2018

Copy link
Copy Markdown
Contributor

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

frangio commented Jan 25, 2018

Copy link
Copy Markdown
Contributor

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
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.

Use Truffle artifacts.require correctly

2 participants