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

Upgrade prettier-solidity #68

Open
wants to merge 1 commit into
base: development
Choose a base branch
from

Conversation

fvictorio
Copy link

Description

Upgrade prettier solidity to its latest version

Checklist
  • Linter status: 100% pass
  • Changes don't break existing behavior
  • Test coverage hasn't decreased
Testing

It only changes the format of the code


Hi everyone! We released a beta version of prettier-solidity this week, so I'm checking which projects are using it to see if this version works fine.

I upgraded the dependency to this latest version and also re-run prettier on your contracts. These caused a lot of changes, but I think it shouldn't be hard to review them with the whitespace diff disabled.

Also: I'm compiling a list of projects that use prettier solidity, would it be ok with you if I include PieDAO on that list?

Thank you!

@MickdeGraaf
Copy link
Member

MickdeGraaf commented Nov 22, 2020

Thank you @fvictorio!

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.

2 participants