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

Remove python 2.7 #60

Merged
merged 8 commits into from
Jun 10, 2024
Merged

Remove python 2.7 #60

merged 8 commits into from
Jun 10, 2024

Conversation

cunla
Copy link
Contributor

@cunla cunla commented May 28, 2024

  • format based on flake8.
  • fix GitHub workflow.
  • remove python 2.7.

jsonpointer.py Outdated Show resolved Hide resolved
@cunla
Copy link
Contributor Author

cunla commented May 28, 2024

@cunla cunla requested a review from stefankoegl May 29, 2024 13:18
@@ -6,3 +6,4 @@ dist
*.swp
doc/_build
*.egg-info
.idea
Copy link
Owner

Choose a reason for hiding this comment

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

What's that? Never heard of it

Copy link
Contributor Author

Choose a reason for hiding this comment

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

jetbrains IDEs create this folder.
This just prevents from adding it accidently.

# Will be parsed by setup.py to determine package metadata
__author__ = 'Stefan Kögl <[email protected]>'
__version__ = '2.4'
__version__ = '3.0.0'
Copy link
Owner

Choose a reason for hiding this comment

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

Any specific reason to go for 3 digits? Is there some standard to follow?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@cunla cunla requested a review from stefankoegl June 2, 2024 13:39
@stefankoegl stefankoegl merged commit f3643ad into stefankoegl:master Jun 10, 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.

3 participants