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

Package testing documentation #398

Merged
merged 8 commits into from
Jul 4, 2023

Conversation

trudeaua21
Copy link
Contributor

@trudeaua21 trudeaua21 commented Jun 16, 2023

Changes

  • Adds documentation for package testing setup tasks, caveats, and configuration options which will be relevant once the package testing PR has been merged into the Unity test runner.

Checklist

  • Verify build works
  • Verify Links between markdown sections work (wasn't able to test without building)
  • Read the contribution guide and accept the
    code of conduct
  • Readme (updated or not needed)
  • Tests (added, updated or not needed)

NOTE: I'm blocked from completing the two remaining checklist items by #397 . Since the contributing guidelines say I should ensure there are no build errors before submitting the PR, I'll leave it in draft status until I can figure out a solution or workaround to my issue.

That said, this PR has all of the changes I planned to make - I'm only leaving it a draft for now as a formality.

Issues have been resolved!

@github-actions
Copy link

Cat Gif

@trudeaua21 trudeaua21 marked this pull request as ready for review June 27, 2023 23:01
@trudeaua21
Copy link
Contributor Author

Resolved my build issues - this should be good for review now!

@davidmfinol davidmfinol enabled auto-merge (squash) July 4, 2023 19:02
@davidmfinol davidmfinol merged commit b631734 into game-ci:main Jul 4, 2023
5 checks passed
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