Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Issue #317 #323

Merged
merged 1 commit into from
Jan 24, 2022
Merged

Issue #317 #323

merged 1 commit into from
Jan 24, 2022

Conversation

ninetyninereds
Copy link
Contributor

Changes

added cwd: projectPath parameter to running of git in hasAnyVersionTags

  • ...

Checklist

  • Read the contribution guide and accept the code of conduct
  • [x ] Readme (updated or not needed)
  • Tests (added, updated or not needed)

@github-actions
Copy link

Cat Gif

@webbertakken
Copy link
Member

Hey @ninetyninereds I saw your comments earlier.

LGTM. Does this work as expected for you?

You could check using game-ci/unity-builder@<specific commit sha>

@ninetyninereds
Copy link
Contributor Author

ninetyninereds commented Jan 24, 2022 via email

@codecov-commenter
Copy link

Codecov Report

Merging #323 (01012f0) into main (018e347) will not change coverage.
The diff coverage is n/a.

❗ Current head 01012f0 differs from pull request most recent head 8e10ed4. Consider uploading reports for the commit 8e10ed4 to get more accurate results
Impacted file tree graph

@@           Coverage Diff           @@
##             main     #323   +/-   ##
=======================================
  Coverage   56.72%   56.72%           
=======================================
  Files          23       23           
  Lines         781      781           
  Branches      151      151           
=======================================
  Hits          443      443           
  Misses        337      337           
  Partials        1        1           
Impacted Files Coverage Δ
src/model/versioning.ts 93.96% <ø> (ø)

@webbertakken webbertakken merged commit 7871734 into game-ci:main Jan 24, 2022
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants