Skip to content

Add VersionSourceSha to output #1618

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

Merged

Conversation

hayhay27
Copy link
Contributor

@hayhay27 hayhay27 commented Mar 1, 2019

VersionSourceSha is sha of baseVersionSource commit in MetadataCalculator

Resolves #1607.

hayhay27 added 2 commits March 1, 2019 16:52
VersionSourceSha is sha of baseVersionSource commit in MetadataCalculator
@arturcic arturcic requested a review from asbjornu March 1, 2019 14:37
Copy link
Member

@asbjornu asbjornu left a comment

Choose a reason for hiding this comment

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

Looks good, but I'm missing at least one test specifically for VersionSourceSha. Think you can add one or more, @hayhay27?

@hayhay27
Copy link
Contributor Author

hayhay27 commented Mar 1, 2019

Looks good, but I'm missing at least one test specifically for VersionSourceSha. Think you can add one or more, @hayhay27?

I thought about special test but many cases cover by existing tests (truly for SemanticVersionBuildMetaData). I not found any test on MetaDataCalculator. I can make them for it and check VersionSourceSha. Other modified tests already check output variables against approved results. Including VersionSourceSha.

@hayhay27
Copy link
Contributor Author

hayhay27 commented Mar 4, 2019

@asbjornu, I look forward to your approve.

@asbjornu asbjornu merged commit debc1fc into GitTools:master Mar 13, 2019
@asbjornu
Copy link
Member

Excellent, thanks for your contribution @hayhay27!

@hayhay27 hayhay27 deleted the feature/add-versionsourcesha-to-output branch March 13, 2019 17:33
@hayhay27
Copy link
Contributor Author

I'm glad, thanks.

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.

2 participants