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

CI: Remove crate publish check #14044

Merged
2 commits merged into from
Apr 28, 2023
Merged

CI: Remove crate publish check #14044

2 commits merged into from
Apr 28, 2023

Conversation

ggwpez
Copy link
Member

@ggwpez ggwpez commented Apr 28, 2023

The check is publish script is broken, therefore now removing as to not block development.
There is another job with name check-crate-publishing in scripts/ci/gitlab/pipeline/publish.yml, so i hope to have removed the correct one 🙈

Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>
Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>
@ggwpez ggwpez requested a review from a team as a code owner April 28, 2023 18:50
@ggwpez ggwpez 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 labels Apr 28, 2023
@paritytech-ci paritytech-ci requested a review from a team April 28, 2023 20:08
@ghost ghost merged commit de80d01 into master Apr 28, 2023
@ghost ghost deleted the oty-remove-publish-crates-ci branch April 28, 2023 23:34
altaua pushed a commit that referenced this pull request May 2, 2023
Originally attempted in #14044. It's been broken for a while and we're
not sure yet how to move forward in the longer term.
altaua pushed a commit that referenced this pull request May 3, 2023
* Restore `check-crates-publishing-pipeline` job

This job was errorneously deleted in lieu of the
`check-crates-publishing` job.

This reverts commit de80d01.

* Don't run `check-crate-publishing` job on prs.

Originally attempted in #14044. It's been broken for a while and we're
not sure yet how to move forward in the longer term.
gpestana pushed a commit that referenced this pull request May 4, 2023
* Fix frame-support feature

Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>

* Remove publishing script

Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>

---------

Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>
gpestana pushed a commit that referenced this pull request May 4, 2023
* Restore `check-crates-publishing-pipeline` job

This job was errorneously deleted in lieu of the
`check-crates-publishing` job.

This reverts commit de80d01.

* Don't run `check-crate-publishing` job on prs.

Originally attempted in #14044. It's been broken for a while and we're
not sure yet how to move forward in the longer term.
nathanwhit pushed a commit to nathanwhit/substrate that referenced this pull request Jul 19, 2023
* Fix frame-support feature

Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>

* Remove publishing script

Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>

---------

Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>
nathanwhit pushed a commit to nathanwhit/substrate that referenced this pull request Jul 19, 2023
* Restore `check-crates-publishing-pipeline` job

This job was errorneously deleted in lieu of the
`check-crates-publishing` job.

This reverts commit de80d01.

* Don't run `check-crate-publishing` job on prs.

Originally attempted in paritytech#14044. It's been broken for a while and we're
not sure yet how to move forward in the longer term.
This pull request was closed.
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants