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

CI: rephrase RUSTFLAGS #13735

Merged
merged 1 commit into from
Mar 28, 2023
Merged

CI: rephrase RUSTFLAGS #13735

merged 1 commit into from
Mar 28, 2023

Conversation

rcny
Copy link
Contributor

@rcny rcny commented Mar 28, 2023

For whatever reason cargo started to receive truncated RUSTFLAGS on master, which led to the RUSTFLAGS inconsistency between master and PRs and further cache invalidation.

This PR reformats RUSTFLAGS in a more accepted way, which brings the flags consistency back.

Resolves https://github.com/paritytech/ci_cd/issues/762.

@rcny rcny 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. labels Mar 28, 2023
@rcny rcny requested a review from a team as a code owner March 28, 2023 11:07
@rcny rcny self-assigned this Mar 28, 2023
@paritytech-ci paritytech-ci requested a review from a team March 28, 2023 11:11
@rcny rcny merged commit dec0369 into master Mar 28, 2023
@rcny rcny deleted the vi/rephase-rustflags branch March 28, 2023 11:41
pgherveou pushed a commit that referenced this pull request Apr 4, 2023
@rcny @pgherveou
CI: rephrase RUSTFLAGS (#13735)
7fa7439
gpestana pushed a commit that referenced this pull request Apr 23, 2023
@rcny @gpestana
CI: rephrase RUSTFLAGS (#13735)
f34444b
@kacperzuk-neti kacperzuk-neti mentioned this pull request Jun 23, 2023
Merged
15 tasks
nathanwhit pushed a commit to nathanwhit/substrate that referenced this pull request Jul 19, 2023
@rcny @nathanwhit
CI: rephrase RUSTFLAGS (paritytech#13735)
81d5fe4
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@alvicsam alvicsam alvicsam approved these changes

@altaua altaua altaua approved these changes

Assignees

@rcny rcny

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.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@rcny @alvicsam @altaua