You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
As the requirements for Core Components have been shifting from the time at which the original guidelines have shifted, this issue is a request to deprecate two related requirements for promotion:
Good unit test coverage measured by community (e.g. 100% or 75% of what’s important)
uses Coverage tool (coveralls or simplecov)
coveralls has been unreliable with regards to the analysis of code coverage analysis in past maintenance contributions, which simplecov has also presented issues regarding the threshold coverage at which each Gem should be maintained.
Removing these requirements would also ensure that all newer contributors need not feel obligated to configure code coverage analysis tools such as simplecov, and can instead focus upon crucial elements of functionality within the code base.
The text was updated successfully, but these errors were encountered:
jrgriffiniii
changed the title
Remove the simplecov requirements for promoting Core Components
Remove the simplecov requirement for promoting Core Components
Jun 22, 2022
I'm curious if anyone can even figure out what simplecov currently reports for hyrax, where it is theoretically configured.
I think few if anyone have been using these reports for some time; as we try to maintain some legacy gems we are running into trouble where simplecov isn't working, easier to remove it than to spend time trying to fix it if it's not providing much value. See samvera/questioning_authority#362 and samvera/browse-everything#401
After reviewing https://github.com/simplecov-ruby/simplecov, I can perhaps also augment this with a proposed set of recommendations for analyzing test coverage and persisting test coverage reports for CircleCI.
As the requirements for Core Components have been shifting from the time at which the original guidelines have shifted, this issue is a request to deprecate two related requirements for promotion:
coveralls
has been unreliable with regards to the analysis of code coverage analysis in past maintenance contributions, whichsimplecov
has also presented issues regarding the threshold coverage at which each Gem should be maintained.Further, there exist cases in which, due to the shift in available labor needed to increase or maintain the recommended 75%-100% threshold, this will render promoting components where required will become impossible. One such example is https://github.com/samvera/active_fedora, where the coverage is currently reported as 65%: https://coveralls.io/github/samvera/active_fedora?branch=main.
Removing these requirements would also ensure that all newer contributors need not feel obligated to configure code coverage analysis tools such as
simplecov
, and can instead focus upon crucial elements of functionality within the code base.The text was updated successfully, but these errors were encountered: