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: bump version for protected branches #27

Merged
merged 2 commits into from
May 8, 2022
Merged

fix: bump version for protected branches #27

merged 2 commits into from
May 8, 2022

Conversation

h1dden-da3m0n
Copy link
Contributor

Description

Sorry for taking so long with this simple fix, but live got in the way and motivation bit hard to build up.
This PR will reintroduce the version bump to the 'changelog' workflow that then gets part of the generated PRs.

Changes

  • deprecate the bump-version workflow
  • re-add bumping to the changelog workflow
  • remove obsolete yq-setup, since its now included in the GitHub Actions Virtual Environments

Issues

  • n/a
  • Matrix Dev chat comment

the auto bump does not work for protected branches, which some plugins have configured
yq is already included in new GitHub Action virtual-environments in a recent enough version
@h1dden-da3m0n h1dden-da3m0n added dependency This PR updates or changes a dependency file ci This PR updates or changes something CI related labels May 8, 2022
@crobibero
Copy link
Member

Just to make sure I'm reading this correctly- this change requires all plugins that use the workflows to have a Directory.Build.props file?

@h1dden-da3m0n
Copy link
Contributor Author

h1dden-da3m0n commented May 8, 2022

Nope its optional. (on porous)
This file would only be needed if you want the version of the compiled assembly to have the version you find in the build.yaml.

However, if we opt to not have it my vote would go for setting the assembly version in ALL plugins to version 0.0.0.0 so the sole source of truth is build.yaml and no confusing version divergence to what is actually reported by the assembly.

@crobibero crobibero merged commit af74ae7 into jellyfin:master May 8, 2022
@h1dden-da3m0n h1dden-da3m0n deleted the fix/bump-version branch May 8, 2022 19:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci This PR updates or changes something CI related dependency This PR updates or changes a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants