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

chore(repo): pivot releases to creating a pr #20

Merged
merged 1 commit into from
Mar 18, 2024

Conversation

rwaskiewicz
Copy link
Contributor

@rwaskiewicz rwaskiewicz commented Mar 15, 2024

What is the current behavior?

see 'new behavior' section

GitHub Issue Number: N/A

What is the new behavior?

as mentioned in #15, it turns out our branch protections will prevent us from pushing to main in a github action.

in the interest of time and to provide some release consistency to the rest of the team, this commmit pivots the existing release-prod workflow to a workflow that will only create a pull request that bumps the version of the project.

in a separate commit/pr, we shall introduce a new workflow to do the actual publish

Documentation

Updated the release steps for this half of the release only

Does this introduce a breaking change?

  • Yes
  • No

Testing

I used 5c099bf to test this PR. I couldn't rename the workflow and run it simultaneously (without adding a push trigger to it). I moved it in the subsequent commit, but it should work just as well once this lands. I opted for this testing method to be able to hit "Run Workflow" in the GH Actions UI, rather than overriding the inputs to the workflow/assuming they worked.

I was able to use the aforementioned commit to run the workflow (logs) to generate a PR - #21

Other information

@rwaskiewicz rwaskiewicz marked this pull request as ready for review March 15, 2024 21:01
@rwaskiewicz rwaskiewicz requested a review from a team as a code owner March 15, 2024 21:01
@rwaskiewicz
Copy link
Contributor Author

Moving back to draft, all my testing info/steps got swallowed 😢

@rwaskiewicz rwaskiewicz marked this pull request as draft March 15, 2024 21:02
@rwaskiewicz rwaskiewicz marked this pull request as ready for review March 18, 2024 13:33
Copy link
Member

@christian-bromann christian-bromann left a comment

Choose a reason for hiding this comment

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

One comment/question

@rwaskiewicz rwaskiewicz added this pull request to the merge queue Mar 18, 2024
@rwaskiewicz rwaskiewicz removed this pull request from the merge queue due to a manual request Mar 18, 2024
as mentioned in #15,
it turns out our branch protections will prevent us from pushing to
`main` in a github action.

in the interest of time and to provide some release consistency to the
rest of the team, this commmit pivots the existing `release-prod`
workflow to a workflow that will only create a pull request that bumps
the version of the project.

in a separate commit/pr, we shall introduce a new workflow to do the
actual publish
@rwaskiewicz rwaskiewicz force-pushed the rw/create-pr-for-release branch from b153c4c to a0fc617 Compare March 18, 2024 20:29
@rwaskiewicz rwaskiewicz added this pull request to the merge queue Mar 18, 2024
Merged via the queue into main with commit e850f43 Mar 18, 2024
3 checks passed
@rwaskiewicz rwaskiewicz deleted the rw/create-pr-for-release branch March 18, 2024 20:33
rwaskiewicz added a commit that referenced this pull request Mar 18, 2024
in #20, this was accidentally left with its old name
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.

3 participants