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

Use Coveralls GitHub Action #2266

Merged
merged 3 commits into from
Jun 15, 2022
Merged

Conversation

duffn
Copy link
Member

@duffn duffn commented Jun 15, 2022

This uses the Coveralls GitHub Action in place of coveralls-reborn as recommended in the gem README https://github.com/tagliala/coveralls-ruby-reborn#github-actions and as discussed here #2265. The parallel test steps are taken from the Action README https://github.com/coverallsapp/github-action

Examples on my fork:

@grape-bot
Copy link

grape-bot commented Jun 15, 2022

1 Message
📖 We really appreciate pull requests that demonstrate issues, even without a fix. That said, the next step is to try and fix the failing tests!

Generated by 🚫 Danger

@dblock
Copy link
Member

dblock commented Jun 15, 2022

Much better! Add a changelog with something like “Fixed code coverage” and we are good to go.

@duffn
Copy link
Member Author

duffn commented Jun 15, 2022

Updated.

@dblock dblock merged commit f5dc7dc into ruby-grape:master Jun 15, 2022
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