Skip to content

Update README: fix Snyk badge#3250

Merged
arkid15r merged 1 commit intovacanza:devfrom
KJhellico:upd-readme-snyk
Feb 3, 2026
Merged

Update README: fix Snyk badge#3250
arkid15r merged 1 commit intovacanza:devfrom
KJhellico:upd-readme-snyk

Conversation

@KJhellico
Copy link
Copy Markdown
Collaborator

@KJhellico KJhellico commented Feb 2, 2026

Proposed change

It seems that Snyk badge is no longer available. So I try to use "Package Health Score" value directly from page (regex = '(class="score-number".*?>)(\d{1,3}/\d{1,3})', replace = '$2').

Generated documentation preview.

Type of change

  • New country/market holidays support (thank you!)
  • Supported country/market holidays update (calendar discrepancy fix, localization)
  • Existing code/documentation/test/process quality improvement (best practice, cleanup, refactoring, optimization)
  • Dependency update (version deprecation/pin/upgrade)
  • Bugfix (non-breaking change which fixes an issue)
  • Breaking change (a code change causing existing functionality to break)
  • New feature (new holidays functionality in general)

Checklist

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Feb 2, 2026

Summary by CodeRabbit

  • Documentation
    • Updated project status badges in README with improved tracking paths and visual styling.

Walkthrough

Updated the README's CI/CD and Snyk badge section by modifying the Snyk badge URL from an advisor-based endpoint to a security-focused package path and adding a color parameter (color=41B5BE) to the CI/CD status badge image.

Changes

Cohort / File(s) Summary
Badge Updates
README.md
Updated Snyk badge URL from advisor-based path to security package path; added color parameter to CI/CD badge.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Suggested reviewers

  • PPsyrius
🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title 'Update README: fix Snyk badge' directly and clearly summarizes the main change—fixing the Snyk badge in the README file.
Description check ✅ Passed The description explains the issue (badge unavailable), the solution (extract Package Health Score from Snyk page), and provides relevant details like the regex pattern and new URL source.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud bot commented Feb 2, 2026

Copy link
Copy Markdown
Collaborator

@arkid15r arkid15r left a comment

Choose a reason for hiding this comment

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

LGTM

@arkid15r arkid15r enabled auto-merge February 2, 2026 23:36
@codecov
Copy link
Copy Markdown

codecov bot commented Feb 2, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (090b159) to head (81e59f9).
⚠️ Report is 1 commits behind head on dev.

Additional details and impacted files
@@            Coverage Diff            @@
##               dev     #3250   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files          307       307           
  Lines        18377     18377           
  Branches      2358      2358           
=========================================
  Hits         18377     18377           

☔ 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.

@arkid15r arkid15r added this pull request to the merge queue Feb 2, 2026
Merged via the queue into vacanza:dev with commit da10b28 Feb 3, 2026
47 of 50 checks passed
@KJhellico KJhellico deleted the upd-readme-snyk branch February 4, 2026 11:08
@arkid15r arkid15r mentioned this pull request Feb 16, 2026
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.

2 participants