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

fix(CI): don't PR against "next" if we can't cleanly cherry pick #6703

Merged
merged 2 commits into from
Oct 14, 2022

Conversation

jtoar
Copy link
Contributor

@jtoar jtoar commented Oct 14, 2022

Fixes what we saw happen in https://github.com/redwoodjs/redwood/actions/runs/3245650941. In this run, even though the fast-forward job couldn't cleanly cherry pick the commit in the squashed PR, the PR step could (somehow), and went ahead and made a PR. Let's just exit if the first step fails.

This PR also adds a skip-cherry-pick flag for PRs that will cleanly cherry pick but that we don't want in the next, like those in the last few commits.

@jtoar jtoar added the release:chore This PR is a chore (means nothing for users) label Oct 14, 2022
@jtoar jtoar changed the title fix: don't pr if can't cherry pick cleanly fix(ci): don't PR against next if the fast-forward isn't clean Oct 14, 2022
@jtoar jtoar changed the title fix(ci): don't PR against next if the fast-forward isn't clean fix(CI): don't PR against "next" if we can't cleanly cherry pick Oct 14, 2022
@jtoar jtoar merged commit 8a107ff into main Oct 14, 2022
@jtoar jtoar deleted the ds-fix-cherry-pick-ci branch October 14, 2022 00:42
@redwoodjs-bot redwoodjs-bot bot added this to the next-release milestone Oct 14, 2022
dac09 added a commit that referenced this pull request Oct 26, 2022
…aching

* 'main' of github.com:redwoodjs/redwood: (244 commits)
  chore(deps): update dependency @replayio/playwright to v0.3.0 (#6735)
  chore: update all contributors
  Update Clerk docs (#6712)
  Update firebase auth docs (#6717)
  Clerk: Simplify web implementation (#6713)
  Add auth decoder to clerk auth setup (#6718)
  Auth: Update firebase setup script (#6716)
  chore: Remove redundant space " " (#6714)
  Update the Clerk setup script and templates (#6710)
  Fix decouple auth related type errors (#6709)
  fix(deps): update dependency css-minimizer-webpack-plugin to v4.2.2 (#6688)
  fix(deps): update dependency @graphql-codegen/cli to v2.13.7 (#6687)
  feat: publish 2nd canary (@next) from release branch (#6505)
  fix: don't pr if can't cherry pick cleanly (#6703)
  fix(dbAuth): add required packages to setup command (#6698)
  Netlify: Enable auth-providers-api and auth-providers-web installation (#6697)
  chore: make misc change to trigger canary publishing (#6695)
  chore: remove private on new packages (#6692)
  chore: run lint fix (#6691)
  Decouple auth (#5985)
  ...
@jtoar jtoar modified the milestones: next-release, v3.3.0 Oct 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release:chore This PR is a chore (means nothing for users)
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

1 participant