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 support for old CKAN versions and Python 2 #244

Merged
merged 2 commits into from
Apr 27, 2023

Conversation

seitenbau-govdata
Copy link
Member

@seitenbau-govdata seitenbau-govdata commented Apr 21, 2023

As announced in #241 we cleaned up the extension of code from CKAN versions lower than 2.9 and Python2.

  • Removed Python2 compatibility (six and py2 ifelses)
  • Removed checks and ifelses for CKAN versions < 2.9
  • Removed pylons support
  • Removed paster commands
  • Cleaned up github actions

@seitenbau-govdata
Copy link
Member Author

The only tests that are currently failing will be fixed with #241.

@seitenbau-govdata seitenbau-govdata marked this pull request as ready for review April 25, 2023 11:05
@seitenbau-govdata seitenbau-govdata force-pushed the drop-support-for-old-versions branch from aad00dc to 8675452 Compare April 25, 2023 11:24
@seitenbau-govdata
Copy link
Member Author

@amercader and @pdelboca The pull request is now ready for review. Many thanks in advance! 😃

@amercader
Copy link
Member

Thanks @seitenbau-govdata ! Can you fix the conflicts so we can merge?

@seitenbau-govdata seitenbau-govdata force-pushed the drop-support-for-old-versions branch from 8675452 to 482bb1b Compare April 27, 2023 11:05
@seitenbau-govdata
Copy link
Member Author

seitenbau-govdata commented Apr 27, 2023

Thanks @seitenbau-govdata ! Can you fix the conflicts so we can merge?

@amercader Done

@amercader amercader merged commit 5faebce into ckan:master Apr 27, 2023
@seitenbau-govdata seitenbau-govdata deleted the drop-support-for-old-versions branch April 27, 2023 13:12
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