Skip to content

chore(dependencies): re-generate uv lockfile to bring in patched Pygments version#166

Merged
lukeroantreeONS merged 1 commit intomainfrom
165-resolve-pygments-dependabot-alert
Apr 9, 2026
Merged

chore(dependencies): re-generate uv lockfile to bring in patched Pygments version#166
lukeroantreeONS merged 1 commit intomainfrom
165-resolve-pygments-dependabot-alert

Conversation

@lukeroantreeONS
Copy link
Copy Markdown
Contributor

@lukeroantreeONS lukeroantreeONS commented Apr 8, 2026

📌 Address Dependabot Alerts

✨ Summary

Dependabot flagged two vulnerabilities in two dependencies, re-generating the uv.lock file was sufficient to bring in the patched versions.

dependencies:

  • pygments
  • cryptography

📜 Changes Introduced

  • chore: re-generated lockfile to bring in patched pygments version (v2.20.0) and patched cryptography version (v46.0.7)

✅ Checklist

Please confirm you've completed these checks before requesting a review.

  • Code passes linting with Ruff
  • Security checks pass using Bandit
  • API and Unit tests are written and pass using pytest
  • Terraform files (if applicable) follow best practices and have been validated (terraform fmt & terraform validate)
  • DocStrings follow Google-style and are added as per Pylint recommendations
  • Documentation has been updated if needed

🔍 How to Test

  • checkout the branch
  • sync your environment with the updated uv.lock file; uv sync
  • confirm the patched version of pygments is installed; uv tree | grep --color=always -E 'pygments|$'

@lukeroantreeONS lukeroantreeONS requested a review from a team as a code owner April 8, 2026 08:29
@lukeroantreeONS lukeroantreeONS linked an issue Apr 8, 2026 that may be closed by this pull request
@github-actions github-actions Bot added the chore label Apr 8, 2026
Copy link
Copy Markdown
Contributor

@rileyok-ons rileyok-ons left a comment

Choose a reason for hiding this comment

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

Passed testing and using uv audit, looks great

@Tom-Owen-ONS
Copy link
Copy Markdown
Contributor

Wonder if you don't need to regenerate the whole lock file if you just do

uv sync --upgrade

Should just update the deps that have available updates rather than requiring a whole lock file rewrite, though the results should be pretty much the same.

@Tom-Owen-ONS
Copy link
Copy Markdown
Contributor

I can't find the dependabot config file (dependabot.yml) to see how it's set up... has it been done another way?

@lukeroantreeONS
Copy link
Copy Markdown
Contributor Author

I can't find the dependabot config file (dependabot.yml) to see how it's set up... has it been done another way?

We haven't set one up yet (although I'll add a backlog ticket for that now), so it's operating on the default behaviour - I think that's just scanning any 'standard' dependency management files (e.g. requirements.txt, poetry.lock, uv.lock, etc.) once a day.

Wonder if you don't need to regenerate the whole lock file if you just do

uv sync --upgrade

Should just update the deps that have available updates rather than requiring a whole lock file rewrite, though the results should be pretty much the same.

That's good to know, thanks - does this make changes just to your local environment, or to the lockfile too (such that they can be committed)?

@Tom-Owen-ONS
Copy link
Copy Markdown
Contributor

That's good to know, thanks - does this make changes just to your local environment, or to the lockfile too (such that they can be committed)?

Yep, it updates the lock file too... I think it's possibly the same as doing uv lock --upgrade but I'm not too sure.

@lukeroantreeONS
Copy link
Copy Markdown
Contributor Author

Thanks both.

@Tom-Owen-ONS I'll try to use that method as an initial attempt to resolve dependabot issues going forward, thanks - I'll merge this as-is for now though.

@lukeroantreeONS lukeroantreeONS merged commit a16eb3f into main Apr 9, 2026
6 checks passed
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.

Resolve Pygments Dependabot Alert

3 participants