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

feat: support private scoped registry on ubuntu #272

Merged

Conversation

nvandessel
Copy link
Contributor

@nvandessel nvandessel commented May 31, 2024

Changes

  • Moves the creation of the .upmconfig.toml previously used for package testing, to outside the scope of package testing to support projects which contain private scoped registries. The caveats remain the same for packages, that right now it is only supported on Linux runners, and only a single private registry is supported.

Related Issues

Related PRs

Successful Workflow Run Link

PRs don't have access to secrets so you will need to provide a link to a successful run
of the workflows from your own repo.

  • I have a successful run on a private repo, my fork fails the existing tests due to missing Unity License. If needed I could alter the main.yml to only run a single test on a Professional license to show it pass, or we could discuss any other options like sharing screenshots of action passing from private repo.

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

@nvandessel
Copy link
Contributor Author

@GabLeRoux curious to get our input on this and the related issue I created/attached. Thanks! 🙏

@nvandessel nvandessel marked this pull request as ready for review May 31, 2024 21:37
@nvandessel
Copy link
Contributor Author

Apologies for my coworkers reviews, I shared the PR with them among some others and they reviewed thinking it was going into our fork 😅

@webbertakken webbertakken merged commit eba7e93 into game-ci:main Jun 3, 2024
1 check passed
@webbertakken
Copy link
Member

@nvandessel
Copy link
Contributor Author

Damn, that was fast. You guys rock 🤘

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.

Support Private Package Registries in Project testing
5 participants