Skip to content

add python version requirement to BUILDING.md#2

Merged
MegaMech merged 3 commits intoMegaMech:patch-7from
briaguya0:patch-1
May 8, 2022
Merged

add python version requirement to BUILDING.md#2
MegaMech merged 3 commits intoMegaMech:patch-7from
briaguya0:patch-1

Conversation

@briaguya0
Copy link

instead of HarbourMasters#256

briaguya0 added 2 commits May 1, 2022 17:33
extract_assets uses match statements which were added in python version 3.10
HarbourMasters#251 removes the 3.10 specific match statements, dropping the python version requirement to 3.6.
@briaguya0
Copy link
Author

briaguya0 commented May 8, 2022

i'd like to change the link to not be a specific windows python version, but instead https://www.python.org/downloads/

the visual studio link is a download page already (https://visualstudio.microsoft.com/vs/community/), so it seems like there's precedent

it'll prevent us from needing to update the link when new python versions are released

any opposition?

edit: the latest python version on the download page is 3.10.4, the current link is 3.10.2 so it's already out of date

@MegaMech
Copy link
Owner

MegaMech commented May 8, 2022

Seems like a good idea to me. Yes, lets do that.

@MegaMech MegaMech merged commit 7ea5f32 into MegaMech:patch-7 May 8, 2022
@briaguya0 briaguya0 deleted the patch-1 branch May 8, 2022 08:46
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