Skip to content
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.

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

ci: fixes for cargo audit #4895

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

ci: fixes for cargo audit #4895

wants to merge 1 commit into from

Conversation

dougch
Copy link
Contributor

@dougch dougch commented Nov 13, 2024

Release Summary:

Resolved issues:

none

Description of changes:

S2n-tls isn't a pure rust project, more work is required before running the cargo audit action.

Call-outs:

Validated on main of my fork

Due to permissions issues, this action does not run in the PR workflow (hence the missed failure)

Testing:

How is this change tested: locally and in my fork

  • Any change to the CI or tests should:
    1. see my fork
    2. The action is currently failing because there is no Cargo.lock in the root directory of the project.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@github-actions github-actions bot added the s2n-core team label Nov 13, 2024
@dougch dougch marked this pull request as ready for review November 13, 2024 23:33
Comment on lines +21 to +23
# Pin the nightly toolchain to prevent breakage.
# This should be occasionally updated.
RUST_NIGHTLY_TOOLCHAIN: nightly-2024-01-01
Copy link
Contributor

Choose a reason for hiding this comment

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

Why do we need to configure a nightly toolchain? Aren't we using stable below?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants