Skip to content

Add production and release profile#1238

Closed
alvicsam wants to merge 5 commits intomasterfrom
as-ad-profiles
Closed

Add production and release profile#1238
alvicsam wants to merge 5 commits intomasterfrom
as-ad-profiles

Conversation

@alvicsam
Copy link
Copy Markdown
Contributor

Needed for fixing errors in command-bot (e.g. #1235)

@alvicsam alvicsam requested a review from ggwpez August 29, 2023 08:15
@alvicsam alvicsam added the R0-no-crate-publish-required The change does not require any crates to be re-published. label Aug 29, 2023
@alvicsam alvicsam requested a review from a team August 29, 2023 08:21
Co-authored-by: Koute <koute@users.noreply.github.com>
@chevdor
Copy link
Copy Markdown
Contributor

chevdor commented Aug 29, 2023

Why removing some of the flags compared to https://github.com/paritytech/polkadot/blob/master/Cargo.toml#L202-L215 ?

Copy link
Copy Markdown
Contributor

@chevdor chevdor left a comment

Choose a reason for hiding this comment

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

Focusing on the production profile, this LGTM once we add the flags we have in the polkadot repo.

@koute
Copy link
Copy Markdown
Contributor

koute commented Aug 29, 2023

Why removing some of the flags compared to https://github.com/paritytech/polkadot/blob/master/Cargo.toml#L202-L215 ?

The release profile has opt-level set to 3 by default so it's not necessary to add that in, and as for the production profile we just added it back. :P

alvicsam and others added 3 commits August 29, 2023 10:56
Co-authored-by: Chevdor <chevdor@users.noreply.github.com>
@bkchr
Copy link
Copy Markdown
Member

bkchr commented Aug 29, 2023

Duplicate of: #1229

@alvicsam alvicsam closed this Aug 29, 2023
@alvicsam alvicsam deleted the as-ad-profiles branch August 29, 2023 09:28
bkchr pushed a commit that referenced this pull request Apr 10, 2024
* fixed storage_parameter_key

* added test for storage_parameter_key
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

R0-no-crate-publish-required The change does not require any crates to be re-published.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants