Skip to content

Conversation

@BDevParity
Copy link
Contributor

No description provided.

@BDevParity BDevParity requested review from a team as code owners January 27, 2026 15:30
@BDevParity BDevParity marked this pull request as draft January 27, 2026 15:33
@paritytech-workflow-stopper
Copy link

All GitHub workflows were cancelled due to failure one of the required jobs.
Failed workflow url: https://github.com/paritytech/polkadot-sdk/actions/runs/21403267005
Failed job name: fmt

echo "STABLE_BRANCH=$STABLE_BRANCH" >> $GITHUB_OUTPUT

# I am calling it like this because we will call post crates workflow after on this one
echo "RELEASE_BRANCH=post-crates-release-$RELEASE_NAME" >> $GITHUB_OUTPUT
Copy link
Contributor

@EgorPopelyaev EgorPopelyaev Jan 30, 2026

Choose a reason for hiding this comment

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

Suggested change
echo "RELEASE_BRANCH=post-crates-release-$RELEASE_NAME" >> $GITHUB_OUTPUT
echo "CRATES_RELEASE_BRANCH=post-crates-release-$RELEASE_NAME" >> $GITHUB_OUTPUT

Copy link
Contributor

Choose a reason for hiding this comment

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

not sure about this name tho, maybe iut make sense to keep it as just RELEASE_BRANCH

echo "Running parity-publish plan..."
parity-publish plan --prdoc prdoc

- name: Save Plan.toml diff
Copy link
Contributor

Choose a reason for hiding this comment

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

this we only need for the patch releases, as a new stable has way to many chnaged crates and comes a as big bang anyway

echo "Would run: parity-publish apply -p --batch-delay 15 --max-concurrent 1 --batch-size 1"
echo ""
echo "Crates that would be published:"
parity-publish apply --print || true
Copy link
Contributor

Choose a reason for hiding this comment

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

there is an actual dry run mode built in parity-poublish if you run parity-publish apply -p -d

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.

3 participants