Bump versions on libraries #10
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
MaggieFero
approved these changes
Aug 17, 2023
phildini
added a commit
that referenced
this pull request
Feb 27, 2024
* Bump versions * Bump back Pillow due to test failure * Bump setuptools
phildini
added a commit
that referenced
this pull request
Feb 27, 2024
* Bump versions * Bump back Pillow due to test failure * Bump setuptools
phildini
added a commit
that referenced
this pull request
Mar 1, 2024
* Add privileges to codeql-analysis.yml Explicitly give codeql-analysis action the security-events: write permission so it still works even when the default GitHub Actions token is set to read-only. * Bump versions on libraries (#10) * Bump versions * Bump back Pillow due to test failure * Bump setuptools * Bump Flower Version We missed Flower in our version upgrades earlier somehow, proposing the upgrade here. * Bump python images * Bump github python versions * Down gto 3.11 * Put back the things I accidentally deleted I seem to have accidentally removed three dependencies while resolving merge conflicts. Putting them back now. * Pylint ignore deprecation of imghdr module for now After discussion of the deprecation of the imghdr module, we want Pylint to ignore it for now and will return to checking out alternatives after Pycon, because we expect there will be some better options by then. * Fix Spacing So I Don't Break The Other Linter Now I broke another linter with my linterignore by not putting a space where expected in a comment. Fixing that here. * Adding a space again Turns out there were TWO spaces the linter wanted that I didn't have! 🤣 This is what I get for reviewing PRs in the GitHub WebUI. * It Also Wants A Line Break Please, Python Formatting, Have I Added Enough Whitespace Yet???? 😭 * Made The Comment Shorter For The Linter Now it's VERY terse, here's hoping this is enough to pass. * Making the line shorter-er I miscounted characters, off by 5. This should actually be short enough now. * Update requirements.txt * Update .pylintrc * Add timeout to requests.get requests.get was missing a timeout; added a timeout of 15 as aligned to the timeout currently used in bookwyrm/utils/isni.py * Fix what I broke; leave the change Looks like I copypasted more than I needed to! Oops. * Update base_activity.py * Update pylint.yml * Update .pylintrc * Update Dockerfile * Update Dockerfile * Update Dockerfile * Update base_activity.py --------- Co-authored-by: Margaret Fero <MaggieFero@users.noreply.github.com>
MaggieFero
pushed a commit
that referenced
this pull request
May 13, 2024
* Bump versions * Bump back Pillow due to test failure * Bump setuptools
MaggieFero
pushed a commit
that referenced
this pull request
May 14, 2024
* Bump versions * Bump back Pillow due to test failure * Bump setuptools
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.