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 and update README #568

Merged
merged 5 commits into from
Apr 6, 2020
Merged

Conversation

johnk-novu
Copy link
Contributor

@johnk-novu johnk-novu commented Apr 2, 2020

Checklist
  • documentation is changed or added (if applicable)

Description of change

Generate and publish sha256 of each "release" file. Update README for usage.

I am not sure if the travis tests will work, or how the files would get published to GitHub releases exactly.

Issue: #447

TODO: I still need to get the dgoss.sha256 generated in the Makefile.

If #567 is merged first, I will resolve the resulting merge conflicts (and vice-versa if this one is merged first).

Copy link
Member

@aelsabbahy aelsabbahy left a comment

Choose a reason for hiding this comment

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

Looks good overall, just not a big fan of cluttering the readme file.

README.md Outdated Show resolved Hide resolved
@johnk-novu johnk-novu changed the title (WIP) add sha256 files and update README add sha256 files and update README Apr 2, 2020
Copy link
Contributor

@pedroMMM pedroMMM left a comment

Choose a reason for hiding this comment

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

@aelsabbahy it's ready for you.

@aelsabbahy
Copy link
Member

Wonder if this hit the travis outage. I don't see a build for this.

I'm going to go ahead and click the "update branch" button in the hopes that it triggers travis.

@aelsabbahy
Copy link
Member

Cool, that worked. The correct files seem to be generated in travis. Hopefully the next release has them

@aelsabbahy aelsabbahy merged commit 408c64a into goss-org:master Apr 6, 2020
@aelsabbahy
Copy link
Member

Thank you for your contribution

BenjaminHerbert pushed a commit to BenjaminHerbert/goss that referenced this pull request May 28, 2020
* add sha256 files and update README

* add target to compute sha256 for dgoss

* empty commit to trigger ci from when github was down

* PR feedback

Co-authored-by: Ahmed Elsabbahy <aelsabbahy@users.noreply.github.com>
@dklimpel dklimpel mentioned this pull request Jul 13, 2024
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants