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(windows): replacing of urls if gitPrivateToken is set #648

Merged
merged 1 commit into from
May 17, 2024

Conversation

kijz
Copy link
Contributor

@kijz kijz commented Apr 25, 2024

Changes

  • Windows: fixed an error in set_gitcredential.ps1 where url. was inside of the string, resulting in errors when building with: gitPrivateToken

Related Issues

  • I think right now there is none

Related PRs

  • I think right now there is none

Successful Workflow Run Link

  • This only affects private repos. Cannot share a workflow run here unfortunately.
  • Comparing the changes to linux might suffice.

Checklist

  • Read the contribution guide and accept the
    code of conduct
  • Docs (If new inputs or outputs have been added or changes to behavior that should be documented. Please make a PR
    in the documentation repo)
  • Readme (updated or not needed)
  • Tests (added, updated or not needed)

Copy link

Cat Gif

@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 36.97%. Comparing base (f2250e9) to head (ca010b2).

❗ Current head ca010b2 differs from pull request most recent head cda084b. Consider uploading reports for the commit cda084b to get more accurate results

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #648   +/-   ##
=======================================
  Coverage   36.97%   36.97%           
=======================================
  Files          77       77           
  Lines        3121     3121           
  Branches      645      645           
=======================================
  Hits         1154     1154           
  Misses       1824     1824           
  Partials      143      143           

Copy link
Member

@davidmfinol davidmfinol 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 to me

@davidmfinol davidmfinol merged commit 461ecf7 into game-ci:main May 17, 2024
2 checks passed
homapf added a commit to homagames/unity-builder that referenced this pull request Aug 2, 2024
* fix(windows): replacing of urls if gitPrivateToken is set (game-ci#648)

* Mac logs to file

* Cloud runners stdout log

* Also log stderr

* Test

* Fix stdout

* Keep ignoreReturnCode

---------

Co-authored-by: Filip Kajzer <filipkajzer@gmail.com>
@webbertakken
Copy link
Member

Released in v4.3.0.

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.

4 participants