Skip to content
New issue

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

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

Already on GitHub? # to your account

ci: Fix upload rust binary action for nightly #637

Merged
merged 3 commits into from
May 21, 2023

Conversation

afnanenayet
Copy link
Owner

The action requires the event to be triggered by a tag, which does not
happen for nightlies. In this case we need to manually specify the ref.

@codecov-commenter
Copy link

codecov-commenter commented May 21, 2023

Codecov Report

Merging #637 (623ad6f) into main (d1764b8) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #637   +/-   ##
=======================================
  Coverage   59.41%   59.41%           
=======================================
  Files          13       13           
  Lines        1461     1461           
=======================================
  Hits          868      868           
  Misses        593      593           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@afnanenayet afnanenayet force-pushed the afnan/fix-upload-cargo-nightly-action branch from a74d1ee to cbec7b8 Compare May 21, 2023 19:06
The action requires the event to be triggered by a tag, which does not
happen for nightlies. In this case we need to manually specify the ref.
@afnanenayet afnanenayet force-pushed the afnan/fix-upload-cargo-nightly-action branch from cbec7b8 to 4f73878 Compare May 21, 2023 19:09
These used to be in the releases before I switched to the new action.
The upload rust action now allows you to specify a specific profile.
@afnanenayet afnanenayet merged commit 0da818c into main May 21, 2023
@afnanenayet afnanenayet deleted the afnan/fix-upload-cargo-nightly-action branch May 21, 2023 19:29
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants