Skip to content
This repository has been archived by the owner on Mar 13, 2023. It is now read-only.

chore: unpin Pre-commit additional_dependencies #734

Merged
merged 1 commit into from
Dec 6, 2022
Merged

Conversation

silasary
Copy link
Collaborator

@silasary silasary commented Dec 5, 2022

What type of pull request is this?

  • Non-breaking code change
  • Breaking code change
  • Documentation change/addition
  • Tests change
  • CI change
  • Other: Pre-commit

Description

Pre-commit doesn't update pinned additional_dependencies, so these are slowly getting more and more out of date.

Given the choice between manually updating these by hand, or unpinning, I'd rather go with the latter.

Changes

  • Removed some outdated pins in .pre-commit-config.yml

Test Scenario(s)

I ran pre-commit run --all-files after unpinning, and flake8 still passed.

Checklist

  • I've formatted my code with Black
  • I've added docstrings to everything I've touched
  • I've ensured my code works on Python 3.10.x
  • I've ensured my code works on Python 3.11.x
  • I've tested my changes
  • I've added tests for my code - if applicable
  • I've updated the documentation - if applicable

Pre-commit doesn't update pinned additional_dependencies, so these are slowly getting more and more out of date.

Given the choice between manually updating these by hand, or unpinning, I'd rather go with the latter.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants