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 missing targetCommitish for github publish [ATLAS-579] #91

Merged

Conversation

Larusso
Copy link
Member

@Larusso Larusso commented Jan 27, 2022

Description

The github publish did not set the target commitish. This is important since we let github create the tag. Without this property github will generate the tag based on the main/default branch which may yield incorrect information.

resolves #89

Changes

  • FIX missing targetCommitish property for github publish

@Larusso Larusso requested a review from Sebu January 27, 2022 13:26
@Larusso Larusso changed the title Fix missing targetCommitish for github publish Fix missing targetCommitish for github publish [ATLAS-579] Jan 27, 2022
@Larusso Larusso force-pushed the fix/missing_target_commitish_for_github_release_3.x branch from 1280810 to 6ef9568 Compare January 28, 2022 08:41
Description
===========

The github publish did not set the target commitish. This
is important since we let github create the tag. Without this
property github will generate the tag based on the main/default branch
which may yield incorrect information.

Changes
=======

* ![FIX] missing `targetCommitish` property for github publish
@Larusso Larusso force-pushed the fix/missing_target_commitish_for_github_release_3.x branch from 6ef9568 to 867d631 Compare January 28, 2022 08:59
@atlas-sonarqube
Copy link

@Larusso Larusso merged commit 915efc4 into release/3.x Jan 28, 2022
@Larusso Larusso deleted the fix/missing_target_commitish_for_github_release_3.x branch January 28, 2022 09:30
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.

1 participant