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

DOC: Specify py2k-compatible version in py3k statement #212

Merged
merged 4 commits into from
Nov 4, 2024

Conversation

DWesl
Copy link

@DWesl DWesl commented Oct 30, 2024

The python>=3.6 requirement at the top of the file dates to early 2020, before v0.1 of MONETIO. Before this, MONET and MONETIO were one package. It looks like the last commit in MONET 2.1.5 merges in the work to split MONET and MONETIO, so someone wanting MONETIO functionality from MONET would need a MONET version prior to that.

The `python>=3.6` requirement at the top of the file dates to early 2020, before v0.1 of MONETIO.  Before this, MONET and MONETIO were one package.  It looks like the last commit in MONET 2.1.5 merges in the work to split MONET and MONETIO, so someone wanting MONETIO functionality from MONET would need a MONET version prior to that.
Copy link
Member

@zmoon zmoon left a comment

Choose a reason for hiding this comment

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

Thanks @DWesl this is useful.

I wonder if we could include your reference to v2.1.5 as a footnote referenced here

https://github.com/noaa-oar-arl/monet/blob/987a1fe77fc8297edadb42710c3d7fc1ab20b616/docs/index.rst?plain=1#L42

to show that technically this was associated with v2.1.5 / noaa-oar-arl/monet#77, though v2.2.0 was the first GitHub Release. I can do this in MONET repo.

docs/installing.rst Outdated Show resolved Hide resolved
docs/installing.rst Outdated Show resolved Hide resolved
DWesl and others added 2 commits November 1, 2024 13:42
Thought I'd caught all of those, thanks for spotting that one.

Co-authored-by: Zachary Moon <[email protected]>
@DWesl
Copy link
Author

DWesl commented Nov 1, 2024

I wonder if we could include your reference to v2.1.5 as a footnote referenced here

https://github.com/noaa-oar-arl/monet/blob/987a1fe77fc8297edadb42710c3d7fc1ab20b616/docs/index.rst?plain=1#L42

to show that technically this was associated with v2.1.5 / noaa-oar-arl/monet#77, though v2.2.0 was the first GitHub Release. I can do this in MONET repo.

Like this?

EDIT: Wrong repo. Should I leave it anyway?

Copy link
Member

@zmoon zmoon left a comment

Choose a reason for hiding this comment

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

I was thinking of MONET, but it makes sense to have it here too.

docs/index.rst Outdated Show resolved Hide resolved
docs/index.rst Outdated Show resolved Hide resolved
@zmoon
Copy link
Member

zmoon commented Nov 1, 2024

Apparently I've spent too long on GitHub.

Co-authored-by: Zachary Moon <[email protected]>
@zmoon
Copy link
Member

zmoon commented Nov 4, 2024

Thanks @DWesl

@zmoon zmoon changed the base branch from stable to develop November 4, 2024 16:12
@zmoon zmoon merged commit ea10e76 into noaa-oar-arl:develop Nov 4, 2024
7 checks passed
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