Skip to content

set urlib above 2.5.0 to address safety lint issue - #108

Merged
fmigneault merged 3 commits into
mainfrom
fix-safety
Jul 23, 2025
Merged

set urlib above 2.5.0 to address safety lint issue#108
fmigneault merged 3 commits into
mainfrom
fix-safety

Conversation

@rbavery

@rbavery rbavery commented Jul 22, 2025

Copy link
Copy Markdown
Collaborator

Description

addresses https://github.com/stac-extensions/mlm/actions/runs/16453896381/job/46505885997

Related Issue

Type of Change

  • 📚 Examples, docs, tutorials or dependencies update;
  • 🔧 Bug fix (non-breaking change which fixes an issue);
  • 🥂 Improvement (non-breaking change which improves an existing feature);
  • 🚀 New feature (non-breaking change which adds functionality);
  • 💥 Breaking change (fix or feature that would cause existing functionality to change);
  • 🔐 Security fix.

Checklist

  • I've read the CONTRIBUTING.md guide;
  • I've updated the CHANGELOG.md with provided changes;
  • I've updated the README.md and/or best-practices.md as applicable with new features;
  • I've updated the code style using make check;
  • I've written tests for all new methods and classes that I created;
  • I've written the docstring in Google format for all the methods and classes that I used.

@fmigneault
fmigneault self-requested a review July 22, 2025 21:56

@fmigneault fmigneault left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I think it applies only to dev dependencies?
I did similar for requests recently: https://github.com/stac-extensions/mlm/pull/107/files

@rbavery
rbavery requested a review from fmigneault July 23, 2025 00:25
@fmigneault
fmigneault merged commit 1d0ad55 into main Jul 23, 2025
10 checks passed
@fmigneault
fmigneault deleted the fix-safety branch July 23, 2025 13:23
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