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

Update test documentation #175

Merged
merged 4 commits into from
Dec 6, 2023
Merged

Update test documentation #175

merged 4 commits into from
Dec 6, 2023

Conversation

milanmlft
Copy link
Member

  • Mention that the INFORMDB_PAT environment variable is required to run the system test
  • Add instructions on how to set up INFORMDB_PAT
  • Remove .env.test and use .env.test.sample instead (as the INFORMDB_PAT will differ per user)

@milanmlft milanmlft requested a review from stefpiatek December 6, 2023 11:30
@milanmlft milanmlft enabled auto-merge (squash) December 6, 2023 11:30
Copy link
Contributor

@stefpiatek stefpiatek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚀

@@ -0,0 +1 @@
.env.test
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

IIRC for CI we've just set an environmental variable for the PAT? Interesting that it doesn't care about the rest of the .env.test

@milanmlft milanmlft merged commit f69aea7 into main Dec 6, 2023
7 checks passed
@milanmlft milanmlft deleted the milanmlft/test-docs branch December 13, 2023 09:50
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.

2 participants