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

Tidy up coveralls integration #1541

Open
2 tasks
mattwynne opened this issue May 12, 2021 · 1 comment
Open
2 tasks

Tidy up coveralls integration #1541

mattwynne opened this issue May 12, 2021 · 1 comment
Labels
🏦 debt Tech debt ⌛ stale Will soon be closed by stalebot unless there is activity 🔧 build Related to build / release process

Comments

@mattwynne
Copy link
Member

Today we discovered a formatter class with no unit tests. There was a bug around this formatter, and the lack of unit tests made it harder to fix.

Measuring code coverage from unit tests would have helped us to catch this problem sooner.

  • fix coverage badge on README
  • add a bot (like in CucumberJS) to comment on PRs about the impact on code coverage
@mattwynne mattwynne added 🏦 debt Tech debt 🔧 build Related to build / release process labels May 12, 2021
@aslakhellesoy aslakhellesoy changed the title Tidy up coverals integration Tidy up coveralls integration May 19, 2021
@stale
Copy link

stale bot commented Apr 14, 2023

This issue has been automatically marked as stale because it has not had recent activity. It will be closed in two months if no further activity occurs.

@stale stale bot added the ⌛ stale Will soon be closed by stalebot unless there is activity label Apr 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🏦 debt Tech debt ⌛ stale Will soon be closed by stalebot unless there is activity 🔧 build Related to build / release process
Projects
None yet
Development

No branches or pull requests

1 participant