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

Feat/clear up revisions stuff #22

Merged

Conversation

bellisk
Copy link
Contributor

@bellisk bellisk commented Oct 25, 2023

No description provided.

@bellisk bellisk changed the base branch from main to feat/update-to-ckan-2.10-python-3 October 25, 2023 17:06
@bellisk bellisk marked this pull request as ready for review October 25, 2023 17:06
In CKAN 2.10, deleted resources are really deleted, so we can't and don't have to control how many deleted revisions we keep around.
@bellisk bellisk force-pushed the feat/clear-up-revisions-stuff branch from d673c2d to cbca16e Compare October 25, 2023 17:07
This is so all our harvesters that contain this option in their config don't immediately fail.
@bellisk bellisk merged commit cc686fe into feat/update-to-ckan-2.10-python-3 Oct 26, 2023
4 checks passed
@bellisk bellisk deleted the feat/clear-up-revisions-stuff branch October 26, 2023 12:24
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