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

Fix black stable version issue with Python 3.8 #1301

Merged

Conversation

gcmoreira
Copy link
Contributor

@gcmoreira gcmoreira commented Oct 8, 2024

@ikelos the problem with black is as follows:

On the other hand:

  • Black latest supported version for Python 3.8 is 24.8.0. See
$ pip install black==24.10.0
ERROR: Ignored the following yanked versions: 21.11b0
ERROR: Ignored the following versions that require a different python version: 24.10.0 Requires-Python >=3.9

ERROR: Could not find a version that satisfies the requirement black==24.10.0 (from versions: 18.3a0, 18.3a1, 18.3a2, 18.3a3, 18.3a4, 18.4a0, 18.4a1, 18.4a2, 18.4a3, 18.4a4, 18.5b0, 18.5b1, 18.6b0, 18.6b1, 18.6b2, 18.6b3, 18.6b4, 18.9b0, 19.3b0, 19.10b0, 20.8b0, 20.8b1, 21.4b0, 21.4b1, 21.4b2, 21.5b0, 21.5b1, 21.5b2, 21.6b0, 21.7b0, 21.8b0, 21.9b0, 21.10b0, 21.11b1, 21.12b0, 22.1.0, 22.3.0, 22.6.0, 22.8.0, 22.10.0, 22.12.0, 23.1a1, 23.1.0, 23.3.0, 23.7.0, 23.9.0, 23.9.1, 23.10.0, 23.10.1, 23.11.0, 23.12.0, 23.12.1, 24.1a1, 24.1.0, 24.1.1, 24.2.0, 24.3.0, 24.4.0, 24.4.1, 24.4.2, 24.8.0)

This PR provides a workaround to keep our workflows running until we raise the minimum supported version to 3.9 or higher.

@gcmoreira
Copy link
Contributor Author

@ikelos It looks like it worked

@gcmoreira gcmoreira marked this pull request as ready for review October 8, 2024 08:00
@ikelos
Copy link
Member

ikelos commented Oct 8, 2024

Awesome find, thanks for the quick fix!

@ikelos ikelos merged commit 4ffaad5 into volatilityfoundation:develop Oct 8, 2024
12 checks passed
gcmoreira added a commit to gcmoreira/volatility3 that referenced this pull request Oct 8, 2024
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