Skip to content
This repository has been archived by the owner on Nov 15, 2023. It is now read-only.

Companion for #13846 #7022

Merged
merged 9 commits into from
Apr 12, 2023
Merged

Conversation

sam0x17
Copy link
Contributor

@sam0x17 sam0x17 commented Apr 6, 2023

@sam0x17 sam0x17 added A0-please_review Pull request needs code review. B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders. D3-trivial 🧸 PR contains trivial changes in a runtime directory that do not require an audit. T1-runtime This PR/Issue is related to the topic “runtime”. labels Apr 6, 2023
@sam0x17 sam0x17 self-assigned this Apr 6, 2023
@sam0x17 sam0x17 requested a review from KiChjang April 6, 2023 20:39
@sam0x17
Copy link
Contributor Author

sam0x17 commented Apr 10, 2023

remaining blocker for this is the Bernhard Schuster's expander crate still relies on syn 1.x in a way that is incompatible with our upgrade elsewhere. Reaching out to him to see if we can get it upgraded. Apparently we own it?

@sam0x17
Copy link
Contributor Author

sam0x17 commented Apr 11, 2023

bot rebase

@paritytech-processbot
Copy link

Rebased

@sam0x17 sam0x17 force-pushed the sam-globally-upgrade-syn-quote branch from 905ba55 to 2d1b463 Compare April 12, 2023 16:19
@sam0x17 sam0x17 force-pushed the sam-globally-upgrade-syn-quote branch from 2d1b463 to d96a8e9 Compare April 12, 2023 16:20
@paritytech-processbot
Copy link

Waiting for commit status.

@paritytech-processbot
Copy link

Merge cancelled due to error. Error: Statuses failed for c3e29f1

@sam0x17
Copy link
Contributor Author

sam0x17 commented Apr 12, 2023

It's really weird because this was green when I merged the main PR 10 mins ago.

@paritytech-cicd-pr
Copy link

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: test-linux-stable
Logs: https://gitlab.parity.io/parity/mirrors/polkadot/-/jobs/2676055

@sam0x17 sam0x17 force-pushed the sam-globally-upgrade-syn-quote branch from 95b7cee to 0b840da Compare April 12, 2023 19:53
@sam0x17
Copy link
Contributor Author

sam0x17 commented Apr 12, 2023

bot merge

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A0-please_review Pull request needs code review. B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders. D3-trivial 🧸 PR contains trivial changes in a runtime directory that do not require an audit. T1-runtime This PR/Issue is related to the topic “runtime”.
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

4 participants