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

Remove scrutinizer #44

Merged
merged 1 commit into from
Feb 21, 2024
Merged

Remove scrutinizer #44

merged 1 commit into from
Feb 21, 2024

Conversation

vjik
Copy link
Member

@vjik vjik commented Feb 21, 2024

Q A
Is bugfix?
New feature?
Breaks BC?
Fixed issues

Copy link

what-the-diff bot commented Feb 21, 2024

PR Summary

  • Modification to .gitattributes file
    The team has updated the .gitattributes file, which is a control file for Git, our version control system. They removed specific instructions related to a tool called Scrutinizer.

  • Removal of .scrutinizer.yml file
    The team has deleted the .scrutinizer.yml file, indicating that we are no longer using the Scrutinizer tool for assessing code quality and coverage.

  • Updates to README.md file
    The instructions in the README.md file - the document that typically includes important information about a project - has been modified. They removed references to Scrutinizer Code Quality and Code Coverage, implying we have switched from using Scrutinizer for these functions. Instead, a new tool, Codecov, is mentioned for Code Coverage, suggesting that this is our new tool for checking how much of our code is covered by our tests.

Copy link

codecov bot commented Feb 21, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (e4db023) 74.03% compared to head (16eb7e7) 74.03%.

Additional details and impacted files
@@            Coverage Diff            @@
##             master      #44   +/-   ##
=========================================
  Coverage     74.03%   74.03%           
  Complexity      130      130           
=========================================
  Files            22       22           
  Lines           312      312           
=========================================
  Hits            231      231           
  Misses           81       81           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@vjik vjik merged commit c72481c into master Feb 21, 2024
18 of 19 checks passed
@vjik vjik deleted the rm-scrutinizer branch February 21, 2024 09:25
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.

1 participant