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

Add .sha256 files to releases #807

Merged
merged 2 commits into from
Sep 4, 2020
Merged

Add .sha256 files to releases #807

merged 2 commits into from
Sep 4, 2020

Conversation

dfreilich
Copy link
Member

Signed-off-by: David Freilich dfreilich@vmware.com

Summary

Add .sha256 files to releases. It worked successfully over here

Documentation

  • Should this change be documented?
    • Yes, see #___
    • No

Related

Related to, but doesn't resolve #799 (still need to add pgp signatures)

@dfreilich dfreilich requested a review from a team as a code owner August 18, 2020 14:18
Copy link
Member

@jromero jromero left a comment

Choose a reason for hiding this comment

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

I see the files being created but I'm not seeing where they are attached as artifacts of a build or release.

@dfreilich dfreilich force-pushed the add-checksums branch 2 times, most recently from c8bdfa9 to c320afa Compare September 2, 2020 09:53
@dfreilich
Copy link
Member Author

@jromero thanks for pointing that out, I rushed this and totally missed that. I switched it to this action, since the default github one isn't being maintained (actions/upload-release-asset#58), and therefore hasn't implemented a files feature (actions/upload-release-asset#4)

image

Testing it out on my fork, I was able to successfully generate a release with the .sha256 files.

Copy link
Member

@jromero jromero left a comment

Choose a reason for hiding this comment

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

ship

Signed-off-by: David Freilich <dfreilich@vmware.com>
* Allows for using a glob for files, ensuring all are uploaded easily

* Rename the zipped dirs to reflect what we want name to be.

Signed-off-by: David Freilich <dfreilich@vmware.com>
@codecov
Copy link

codecov bot commented Sep 3, 2020

Codecov Report

Merging #807 into main will increase coverage by 3.96%.
The diff coverage is 77.78%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #807      +/-   ##
==========================================
+ Coverage   73.52%   77.48%   +3.96%     
==========================================
  Files          75       81       +6     
  Lines        5135     4657     -478     
==========================================
- Hits         3775     3608     -167     
+ Misses       1043      726     -317     
- Partials      317      323       +6     
Flag Coverage Δ
#os_linux 77.48% <77.78%> (+1.46%) ⬆️
#os_macos 73.66% <71.22%> (+1.75%) ⬆️
#os_windows 100.00% <ø> (+28.18%) ⬆️
#unit 77.48% <77.78%> (+3.96%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

@jromero jromero added this to the 0.14.0 milestone Sep 4, 2020
@jromero jromero added the type/chore Issue that requests non-user facing changes. label Sep 4, 2020
@jromero jromero merged commit 26c89d3 into main Sep 4, 2020
@jromero jromero deleted the add-checksums branch September 4, 2020 03:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type/chore Issue that requests non-user facing changes.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Publish checksums
2 participants