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

Add Odin Tests #3195

Merged
merged 2 commits into from
Aug 22, 2023
Merged

Add Odin Tests #3195

merged 2 commits into from
Aug 22, 2023

Conversation

rzuckerm
Copy link
Collaborator

Congrats on taking the first step to contributing to the Sample Programs repository maintained by The Renegade Coder!
For simplicity, please make sure that your pull request includes one and only one contribution.

Please fill one of the sections below as applicable.
Please also add any other relevant information to the Notes section at the bottom.
You may delete or just ignore any other sections.
For more information please refer to our contributing documentation

I Am Adding New Tests for a Language

  • I fixed Add Odin Testing #3194
  • I named the pull request using Add {LANGUAGE} Tests format
  • I added a testinfo.yml files (see [contributing documentation][contributing-new-language])
    • I used an officially supported docker image or one that I personally trust
  • I verified all tests are passing

Other Notes

I had to change "Hello World" sample to get the tests to pass.

@rzuckerm rzuckerm added enhancement Any code that improves the repo tests Testing related labels Aug 14, 2023
@rzuckerm rzuckerm marked this pull request as ready for review August 15, 2023 00:07
@rzuckerm rzuckerm merged commit 40ae7cd into TheRenegadeCoder:main Aug 22, 2023
6 checks passed
@rzuckerm rzuckerm deleted the add-tests-odin branch August 22, 2023 12:32
github-actions bot pushed a commit that referenced this pull request Aug 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Any code that improves the repo tests Testing related
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add Odin Testing
1 participant