8000 Add Github workflow file by MarkKremer · Pull Request #124 · gopxl/beep · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Add Github workflow file #124

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 13 commits into from
Oct 27, 2023
Merged

Add Github workflow file #124

merged 13 commits into from
Oct 27, 2023

Conversation

MarkKremer
Copy link
Contributor
@MarkKremer MarkKremer commented Oct 25, 2023 •
edited
Loading
  • Added Github Actions to build the packages, run gofmt, go test and upload the coverage to coveralls.
  • Added a build status & code coverage badge in readme.

Do this before merging

  1. Go to coveralls.io
  2. Login or create an account. Easiest is to use Github as you'll probably need to connect to it anyway.
  3. Enable the Beep repo. You'll need sufficient access rights to do this.
  4. Take note of the token.
  5. Add the token as a secret to the Github Actions under under Security → Secrets and variables → Actions → Secrets → New repository secret. Give it the name COVERALLS_REPO_TOKEN.

image

  1. Repeat for all other gopxl repo's. I'll make PR's for them too.

@MarkKremer MarkKremer marked this pull request as ready for review October 25, 2023 11:01
@MarkKremer MarkKremer requested a review from a team October 25, 2023 11:01
@duysqubix
Copy link
Contributor

@MarkKremer

Okay sounds great. Done.. secrets added to beep

@duysqubix duysqubix closed this Oct 27, 2023
@duysqubix duysqubix reopened this Oct 27, 2023
@duysqubix duysqubix merged commit c64e515 into main Oct 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@MarkKremer @duysqubix
0