Skip to content

test codacy ruff#2684

Closed
valeriupredoi wants to merge 3 commits intomainfrom
test_codacy_ruff
Closed

test codacy ruff#2684
valeriupredoi wants to merge 3 commits intomainfrom
test_codacy_ruff

Conversation

@valeriupredoi
Copy link
Contributor

Test PR where we can test all the bits and bobs related to the new codacy-ruff integration and fix things all along. Ruff is now integrated and configured in our Codacy, but there may be issues.

@codecov
Copy link

codecov bot commented Mar 5, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 95.29%. Comparing base (2cb0787) to head (036195a).
⚠️ Report is 96 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2684   +/-   ##
=======================================
  Coverage   95.29%   95.29%           
=======================================
  Files         260      260           
  Lines       15357    15357           
=======================================
  Hits        14635    14635           
  Misses        722      722           

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@valeriupredoi
Copy link
Contributor Author

first hurdle - it runs Ruff inside Codacy static analysis! And it picks up the correct configuration file too 🥳

@bouweandela
Copy link
Member

Nice! Could you try introducing some issues so we can see if they're picked up? e.g. add some random spaces somewhere

@valeriupredoi
Copy link
Contributor Author

@valeriupredoi
Copy link
Contributor Author

@ESMValGroup/technical-lead-development-team codacy-ruff works well at first glance, but if you'd like to stress-test it, please add your wildest linting nightmares here, see if they're picked up and reported 🍺

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants