-
Notifications
You must be signed in to change notification settings - Fork 2
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 existing, add more tests and make them run with Github actions #28
Conversation
DAILY_READ_FETCH_FROM_NGIS = true | ||
DAILY_READ_FETCH_FROM_SNPSEQ = true | ||
DAILY_READ_FETCH_FROM_UGC = true | ||
#DAILY_READ_LOG_LOCATION='tests/log_location' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You want to keep this comment here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe as a reminder for future tests? I can get rid of it too.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice! Great to keep developing the tests!
With the current setup, Github actions run the tests on every push and pull request.
Also moved STATUS_PRIORITY and STATUS_PRIORITY_REV to config