Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Reduce the need for dependabot PRs #1078

Merged

Conversation

MarkZH
Copy link
Collaborator

@MarkZH MarkZH commented Jan 12, 2025

Type of pull request:

  • Bug fix
  • Feature
  • Other

Description:

Allow for the installation of compatible upgraded dependencies without the need for a PR. The compatibility requirement (~=) allows for minor version upgrades that should be backwards compatible according to semantic versioning. Major version upgrades (for example, chess 1.11.1 to chess 2.0) still require a PR since they will more than likely need changes to lichess-bot code.

The user can upgrade to the latest compatible dependencies by running pip install --upgrade -r requirements.txt at any time.

Related Issues:

Checklist:

  • I have read and followed the contribution guidelines.
  • I have added necessary documentation (if applicable).
  • The changes pass all existing tests.

Screenshots/logs (if applicable):

Allow for the installation of compatible upgraded dependencies without
the need for a PR. The compatibility requirement (~=) allows for minor
version upgrades that should be backwards compatible according to
semantic versioning. Major version upgrades (for example, chess 1.11.1
to chess 2.0) still require a PR since they will more than likely need
changes to lichess-bot code.

The user can upgrade to the latest compatible dependencies by running
`pip install --upgrade -r requirements.txt` at any time.
@AttackingOrDefending AttackingOrDefending merged commit 456cd33 into lichess-bot-devs:master Jan 12, 2025
23 checks passed
@MarkZH MarkZH deleted the reduce-dependency-prs branch January 12, 2025 12:47
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