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

Rollout12 🚢 #30

Merged
merged 8 commits into from
Dec 2, 2024
Merged

Rollout12 🚢 #30

merged 8 commits into from
Dec 2, 2024

Conversation

ivy-rew
Copy link
Member

@ivy-rew ivy-rew commented Nov 28, 2024

Migrates projects of the axonivy-market org into the new release/12.0 world.

These migrations come with the favour of:

  • applying changes to repos consistently
  • clearly separating manual commits from manually done work
  • giving you a lot of speed

- making development easier as no token passing is required (already nicely solved by gh)
@ivy-rew ivy-rew marked this pull request as ready for review November 29, 2024 10:50
@ivy-rew
Copy link
Member Author

ivy-rew commented Nov 29, 2024

Hey @quanpham-axonivy 👋

I saw you filing Release12 migration PR on some projects, this repo has the power to almost fully automate this for you.
Giving you rest from tedious repetitive work and allowing you to be watchful on real changes that may become relevant.
I think it would be best if you continue further migrations throughout the axonivy-market using this script.

If you need help to actually run it; don't hesitate to ask for help.

@ivy-rew
Copy link
Member Author

ivy-rew commented Nov 29, 2024

Comment on lines +48 to +49
env:
GH_TOKEN: ${{ secrets.GITHUB_TOKEN }}
Copy link
Member Author

Choose a reason for hiding this comment

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

this workflow doesn't really run by now ... as we do not have a single technical octopus (market-org) user that is allowed to file PRs on all repos. I'd vote to introduce one somewhen ...
but for now the pipeline must be run manually:
you may check it out on a simply run ./raise-all-market-products.sh 12.0.0-SNAPSHOT deepl-connector

Copy link
Contributor

@nqhoan-axonivy nqhoan-axonivy left a comment

Choose a reason for hiding this comment

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

Thank you so much!
Looks amazing script 🥇

@nqhoan-axonivy nqhoan-axonivy merged commit 784de8e into master Dec 2, 2024
@nqhoan-axonivy nqhoan-axonivy deleted the rollout12 branch December 2, 2024 01:26
@quanpham-axonivy
Copy link
Contributor

Thanks a lot @ivy-rew, this is greattt 🔥

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