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

ci: Remove all CI except docs on EOL branches (v10) #3483

Merged
merged 1 commit into from
May 4, 2024

Conversation

wolfgangwalther
Copy link
Member

@wolfgangwalther wolfgangwalther commented May 4, 2024

With #3113 in mind, we are going to keep some EOL-branches (currently v10 and v11) around for docs updates, but won't ever push bugfixes to them anymore. With renovate set up to update those older branches, too, this creates two problems:

  • a considerable number of PRs to update CI dependencies to back-branches, too
  • possibly failing builds, which can only be fixed by bigger changes to CI (e.g. FreeBSD builds/CI #2555)

While it's good to have working CI for docs and have renovate make sure the dependencies for this stay up2date - we don't need everything else. Thus, I propose to just remove all CI workflows except docs from the inactive, docs-only back-branches. This means:

  • Do this PR
  • Do the same against v11 (ci: Remove all CI except docs on EOL branches (v11) #3484)
  • Do nothing against v12, because this is the last release branch, for which we might still want to release patch releases
  • Going forward, whenever we release a new major, do the same on old major branches.

This also gives us much faster CI for those back branches and takes fewer resources for nothing.

This would also supersede #3459, at least for the v10 and v11 branches. We'd need to look into v12 again.

@wolfgangwalther wolfgangwalther changed the title ci: Remove all CI except docs on EOL branches ci: Remove all CI except docs on EOL branches (v10) May 4, 2024
@wolfgangwalther
Copy link
Member Author

This would also supersede #3459, at least for the v10 and v11 branches. We'd need to look into v12 again.

v12 doesn't have the code checking for rel- branches anymore, so doesn't need to be adjusted.

@wolfgangwalther wolfgangwalther merged commit 25a4250 into PostgREST:v10 May 4, 2024
5 checks passed
@wolfgangwalther wolfgangwalther deleted the remove-ci-v10 branch May 4, 2024 11:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

1 participant