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 typo in track-merge-target and track-merge-message #4041

Merged
merged 3 commits into from
May 24, 2024

Conversation

DSpirit
Copy link
Contributor

@DSpirit DSpirit commented May 24, 2024

Fix typo in track-merge-target and track-merge-message

Description

Due to the default configuration of the GitHubFlow/v1 having a typo, the GitVersion task fails with an unkown property.
Could not build the configuration instance because following exception occurred: 'Property 'tracks-merge-target' not found on type 'GitVersion.Configuration.BranchConfiguration'.' Please ensure that the /overrideconfig parameters are correct and the configuration file is in the correct format.

Related Issue

https://github.com/GitTools/GitVersion/releases/tag/6.0.0-rc.1

Motivation and Context

Fix the GitVersion task for GitHubFlow/v1.

How Has This Been Tested?

Screenshots (if appropriate):

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

@arturcic arturcic added this to the 6.x milestone May 24, 2024
@arturcic arturcic merged commit 3c7c22a into GitTools:main May 24, 2024
154 checks passed
Copy link
Contributor

mergify bot commented May 24, 2024

Thank you @DSpirit for your contribution!

@arturcic arturcic modified the milestones: 6.x, 6.0.0-rc.2 Jul 16, 2024
@arturcic
Copy link
Member

🎉 This issue has been resolved in version 6.0.0-rc.2 🎉
The release is available on:

Your GitReleaseManager bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants