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

Update to polkadot-v1.0.0 #149

Merged
merged 8 commits into from
Aug 3, 2023
Merged

Update to polkadot-v1.0.0 #149

merged 8 commits into from
Aug 3, 2023

Conversation

1xstj
Copy link
Contributor

@1xstj 1xstj commented Aug 3, 2023

No description provided.

Copy link
Collaborator

@shekohex shekohex left a comment

Choose a reason for hiding this comment

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

Looks good, but I would say to regenerate the contracts and runtime and do not run cargo fmt on them, they are already formatted with another tool.

proposals/src/header.rs Show resolved Hide resolved
@1xstj 1xstj requested a review from shekohex August 3, 2023 10:58
@shekohex
Copy link
Collaborator

shekohex commented Aug 3, 2023

@1xstj could you also please change the CI for formatting to use rust nightly toolchain instead of the stable one? since we ignore all the generated files from the formatting

@1xstj
Copy link
Contributor Author

1xstj commented Aug 3, 2023

@1xstj could you also please change the CI for formatting to use rust nightly toolchain instead of the stable one? since we ignore all the generated files from the formatting

not sure I follow, the CI failure is becuase we dont format the files in /contracts/protocol-solidity, how will nightly/stable change this? 🤔 Will followup in another PR so that the other PRs depending on this is not blocked

@1xstj 1xstj enabled auto-merge (squash) August 3, 2023 12:55
@1xstj 1xstj disabled auto-merge August 3, 2023 12:55
@1xstj
Copy link
Contributor Author

1xstj commented Aug 3, 2023

Ran cargo fmt again, cannot merge PR without all checks being green

@1xstj 1xstj merged commit a960eaf into main Aug 3, 2023
5 checks passed
@1xstj 1xstj deleted the polkadot-v1.0.0 branch August 3, 2023 14:07
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