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 the coveralls Github action #103

Merged
merged 1 commit into from
Oct 23, 2024
Merged

Use the coveralls Github action #103

merged 1 commit into from
Oct 23, 2024

Conversation

bmerry
Copy link
Contributor

@bmerry bmerry commented Oct 22, 2024

Also use it with parallel flags, which will hopefully stop the MacOS coverage builds from failing.

Also use it with parallel flags, which will hopefully stop the MacOS
coverage builds from failing.
@coveralls
Copy link

Coverage Status

coverage: 94.354% (+1.6%) from 92.721%
when pulling 5089f17 on coveralls-action
into 59b8e55 on master.

Copy link
Contributor

@amishatishpatel amishatishpatel left a comment

Choose a reason for hiding this comment

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

Apologies for the delay, I had to read up on GitHub Actions syntax/keywords. Looks fantastic, as always. I have to ask before explicit Approval (genuinely, because I haven't dabbled in the Actions before)

  • How have you confirmed this change works?

@bmerry
Copy link
Contributor Author

bmerry commented Oct 23, 2024

Apologies for the delay, I had to read up on GitHub Actions syntax/keywords. Looks fantastic, as always. I have to ask before explicit Approval (genuinely, because I haven't dabbled in the Actions before)

* How have you confirmed this change works?

Yep: https://coveralls.io/builds/70463282

@bmerry bmerry merged commit 7f3b260 into master Oct 23, 2024
26 checks passed
@bmerry bmerry deleted the coveralls-action branch October 23, 2024 08:29
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