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.

Sign up for GitHub

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

Jump to bottom

Purge PY2 + CKAN 2.8 #33

Merged
merged 7 commits into from
Sep 2, 2022
Merged

Purge PY2 + CKAN 2.8 #33

merged 7 commits into from
Sep 2, 2022

Conversation

nickumia-reisys
Copy link
Contributor

Related to

Changes:

  • Remove CKAN 2.8 references, default to 2.9 in necessary places
  • Remove Python 2 references/dependencies
  • Remove pylons/paster references

@nickumia-reisys
cleanup: remove all py2 references (first pass)
ce6a475 
- Remove reference to 'six'
- Remove reference to CKAN 2.8
- Default to CKAN 2.9 (but leave modularity to call a different version of CKAN)
- Remove PY2 requirements
- Consolidate plugin.py
@nickumia-reisys
cleanup: remove github action for 2.8
4d9859b
@nickumia-reisys
lint: remove unused imports
cb6b5b8
@nickumia-reisys
lint: make more restrictive
7e3bddc 
- Upgrade github action to python 3.8 lint
- Remove select/excludes
@nickumia-reisys
fix: how did lint not catch this?
01ab2f1
@nickumia-reisys
fix: well, this seems obvious after the fact
ebd6b31 
NO MORE MIXINS.
@nickumia-reisys nickumia-reisys changed the title Purge py2 Purge PY2 + CKAN 2.8 Sep 2, 2022
@nickumia-reisys nickumia-reisys requested a review from a team September 2, 2022 19:48
btylerburton
btylerburton approved these changes Sep 2, 2022
View reviewed changes
Copy link
Contributor

@btylerburton btylerburton left a comment

Choose a reason for hiding this comment

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

🐍 🔥

@btylerburton btylerburton merged commit 3d0958b into main Sep 2, 2022
@btylerburton btylerburton deleted the purge-py2 branch September 2, 2022 20:00
@nickumia-reisys nickumia-reisys mentioned this pull request Sep 13, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@btylerburton btylerburton btylerburton approved these changes

Assignees
No one assigned
Labels
None yet
Projects
data.gov team board
Archived in project
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@nickumia-reisys @btylerburton