Skip to content

feat(jans-cedarling): Optimize Cedar libraries size #11281

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 1 commit into from
Apr 24, 2025
Merged

feat(jans-cedarling): Optimize Cedar libraries size #11281

merged 1 commit into from
Apr 24, 2025

Conversation

olehbozhok
Copy link
Contributor

@olehbozhok olehbozhok commented Apr 23, 2025 •
edited
Loading

Prepare

Description

Updated build profile for release version to optimaze size

Target issue

link

closes #11280

Implementation Details

Updated Cargo.toml

[profile.release]
strip = "symbols"
debug-assertions = false
lto = true
opt-level = "z"
codegen-units = 1
overflow-checks = false

Test and Document the changes

  • Static code analysis has been run locally and issues have been fixed
  • Relevant unit and integration tests have been added/updated
  • Relevant documentation has been updated if any (i.e. user guides, installation and configuration guides, technical design docs etc)

Please check the below before submitting your PR. The PR will not be merged if there are no commits that start with docs: to indicate documentation changes or if the below checklist is not selected.

  • I confirm that there is no impact on the docs due to the code changes in this PR.

@olehbozhok
feat(jans-cedarling): update optimization profile for release build
8c50bfb 
Signed-off-by: Oleh Bozhok <olehbozhok@gmail.com>
@olehbozhok olehbozhok self-assigned this Apr 23, 2025
@mo-auto
Copy link
Member

mo-auto commented Apr 23, 2025

🎉 Snyk checks have passed. No issues have been found so far.

✅ security/snyk check is complete. No issues have been found. (View Details)

@mo-auto mo-auto added comp-jans-cedarling Touching folder /jans-cedarling kind-feature Issue or PR is a new feature request labels Apr 23, 2025
@olehbozhok olehbozhok enabled auto-merge (squash) April 23, 2025 19:51
@olehbozhok olehbozhok merged commit 538dc4d into main Apr 24, 2025
2 checks passed
@olehbozhok olehbozhok deleted the jans-cedaling-issue-11280 branch April 24, 2025 07:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@SafinWasi SafinWasi SafinWasi approved these changes

@rmarinn rmarinn rmarinn approved these changes

@nynymike nynymike Awaiting requested review from nynymike

Assignees

@olehbozhok olehbozhok

Labels
comp-jans-cedarling Touching folder /jans-cedarling kind-feature Issue or PR is a new feature request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

feat(jans-cedarling): Optimize Cedar libraries size
4 participants
@olehbozhok @mo-auto @SafinWasi @rmarinn