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.

Sign up for GitHub

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

Jump to bottom

Core: update dependencies #3477

Merged
merged 1 commit into from
Jun 6, 2024
Merged

Core: update dependencies #3477

merged 1 commit into from
Jun 6, 2024

Conversation

black-sliver
Copy link
Member

jellyfish - bug fix - all commits
jinja2 - security update (we don't use that feature) - changelog
schema - only fixes - changelog
platformdirs - android fixes - all commits
cerifi - regular update
cython - a lot, but we have tests - changelog
orjson - adds AVX-512 on linux, bug fixes - changelog
typing_extensions - changelog

@github-actions github-actions bot added affects: core Issues/PRs that touch core and may need additional validation. waiting-on: peer-review Issue/PR has not been reviewed by enough people yet. labels Jun 5, 2024
@Exempt-Medic Exempt-Medic added the is: maintenance Regular updates to requirements and utilities that do not fix bugs or change/add features. label Jun 6, 2024
Berserker66
Berserker66 approved these changes Jun 6, 2024
View reviewed changes
Copy link
Member

@Berserker66 Berserker66 left a comment

Choose a reason for hiding this comment

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

I tend to test these locally already as I run update all every now and then, and at least so far nothing broke. Did not test this exact set for all AP function, of course.

@black-sliver
Copy link
Member Author

I typically have a random mix of dependency versions, so I like checking out the changelog to see if we should test something specifically (beyond unit tests and regular use).

@black-sliver black-sliver merged commit 808f2a8 into ArchipelagoMW:main Jun 6, 2024
13 checks passed
@black-sliver black-sliver deleted the update-core-dependencies branch June 6, 2024 17:49
wu4 pushed a commit to wu4/Archipelago that referenced this pull request Jun 6, 2024
@black-sliver @wu4
Core: update dependencies (ArchipelagoMW#3477)
7383d40
agilbert1412 pushed a commit to agilbert1412/Archipelago that referenced this pull request Jun 13, 2024
@black-sliver @agilbert1412
Core: update dependencies (ArchipelagoMW#3477)
4b47c6c
jnschurig pushed a commit to Tranquilite0/Archipelago-SoulBlazer that referenced this pull request Jun 13, 2024
@black-sliver
Core: update dependencies (ArchipelagoMW#3477)
de3c2a7
sflavelle pushed a commit to sflavelle/Archipelago-tgc that referenced this pull request Jun 20, 2024
@black-sliver @sflavelle
Core: update dependencies (ArchipelagoMW#3477)
7fd442c
qwint pushed a commit to qwint/Archipelago that referenced this pull request Jun 24, 2024
@black-sliver @qwint
Core: update dependencies (ArchipelagoMW#3477)
820c65c
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Berserker66 Berserker66 Berserker66 approved these changes

Assignees
No one assigned
Labels
affects: core Issues/PRs that touch core and may need additional validation. is: maintenance Regular updates to requirements and utilities that do not fix bugs or change/add features. waiting-on: peer-review Issue/PR has not been reviewed by enough people yet.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@black-sliver @Berserker66 @Exempt-Medic