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

Versioning with GitHub releases #236

Merged
merged 9 commits into from
Aug 19, 2024
Merged

Conversation

jkoenig134
Copy link
Member

@jkoenig134 jkoenig134 commented Aug 19, 2024

Readiness checklist

  • I added/updated tests.
  • I ensured that the PR title is good enough for the changelog.
  • I labeled the PR.

Description

We will now publish our runtime using GitHub Releases, this has multiple effects on the work with this repository:

  1. as discussed IQL was moved to another repo again as it doesn't need to get published along with the runtime: https://github.com/nmshd/iql

  2. we do not upgrade versions of libs anymore in this Repository, instead we do our work and then release if we want using tags (created by gh releases)

  3. We have to label our pull requests now. Additionally we do not use Scopes for PR titles anymore

    Feature/A cool new Feature becomes A cool new Feature with the Label Feature

    -> please keep in mind that the labels ci, chore, dependencies, refactor and test will not occur in the changelog, if you want your feature to be included use another one.

@jkoenig134 jkoenig134 changed the title Ci/versioning with GitHub releases Versioning with GitHub releases Aug 19, 2024
@jkoenig134 jkoenig134 added the ci Continuous Integration related stuff label Aug 19, 2024
Copy link

codecov bot commented Aug 19, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

.ci/publishNpm.sh Show resolved Hide resolved
.ci/publishNpm.sh Show resolved Hide resolved
.ci/publishNpm.sh Show resolved Hide resolved
@jkoenig134 jkoenig134 marked this pull request as ready for review August 19, 2024 13:15
@jkoenig134
Copy link
Member Author

@Magnus-Kuhn & @britsta make sure to read the description and update the titles and labels of your open PRs!

@jkoenig134
Copy link
Member Author

@stnmtz FYI!

Copy link
Contributor

@Milena-Czierlinski Milena-Czierlinski left a comment

Choose a reason for hiding this comment

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

Looks reasonable to me, but I wouldn't consider myself to be an expert in the .ci and .github stuff. @jkoenig134 Just in case you want to wait for a more in depth review on that. Otherwise, if anything should arise we can fix it agily. :D

@jkoenig134 jkoenig134 merged commit af076e5 into main Aug 19, 2024
16 checks passed
@jkoenig134 jkoenig134 deleted the ci/versioning-with-github-releases branch August 19, 2024 15:47
@jkoenig134
Copy link
Member Author

Thanks @Milena-Czierlinski, lets be agile!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci Continuous Integration related stuff
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants