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): stub automated production releases #472

Closed
wants to merge 3 commits into from

Conversation

rwaskiewicz
Copy link
Contributor

@rwaskiewicz rwaskiewicz commented Feb 16, 2024

Pull request checklist

Please check if your PR fulfills the following requirements:

  • Docs have been reviewed and added / updated if needed (for bug fixes / features)
  • Build (npm run build) was run locally and any changes were pushed
  • Tests (npm test) were run locally and passed
  • Prettier (npm run prettier) was run locally and passed

Pull request type

Please check the type of change your PR introduces:

  • Bugfix
  • Feature
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • Documentation content changes
  • Other (please describe):

What is the current behavior?

this project is manually published to npm. this is the first step in resolving that.

GitHub Issue Number: N/A

What is the new behavior?

this commit creates the initial pass at automated production releases
for the repo. it is capable of:

  1. building stencil store
  2. versioning the library based on user input
  3. logging the gneerated change to the console
  4. generating and pushing a commit to a branch

the intent of this commit is to validate some of the mechanics of user
input before we start attempting to publish. the plan is for that to
come in a later pr.

unknown: it unknown if additional branch protection rules will hinder
us from pushing to main once we try to do this "for real". we had
some difficulty here with stencil-core, and may have it again here.
we'll merge this to verify the input works as expected. we can try to
derisk this by setting up similar branch protection rules on a dummy
branch and proceed from there

Does this introduce a breaking change?

  • Yes
  • No

Testing

Prior to ec48da6, I was able to use push events to verify everything passed/worked as expected. See https://github.com/ionic-team/stencil-store/actions/runs/7936243883 (note - it'll say 'dev build', I fixed that in 9fe4da1)

Other information

this commit creates the initial pass at automated production releases
for the repo. it is capable of:
1. building stencil store
2. versioning the library based on user input
3. logging the gneerated change to the console
4. generating and pushing a commit to _a_ branch

the intent of this commit is to validate some of the mechanics of user
input before we start attempting to publish. the plan is for that to
come in a later pr.

unknown: it unknown if additional branch protection rules will hinder
us from pushing to `main` once we try to do this "for real". we had
some difficulty here with stencil-core, and may have it again here.
we'll merge this to verify the input works as expected. we can try to
derisk this by setting up similar branch protection rules on a dummy
branch and proceed from there
@rwaskiewicz rwaskiewicz marked this pull request as ready for review February 16, 2024 21:24
@rwaskiewicz rwaskiewicz requested a review from a team as a code owner February 16, 2024 21:24
@rwaskiewicz rwaskiewicz changed the title Rw/prod release flow chore(repo): stub automated production releases Feb 16, 2024
@rwaskiewicz
Copy link
Contributor Author

Closing, learned some things doing this for the playwright package that I'd like to pivot to

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.

1 participant