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

Combine mysql and sqlite tests into a single job #558

Merged
merged 2 commits into from
May 8, 2024

Conversation

amstilp
Copy link
Contributor

@amstilp amstilp commented May 8, 2024

This way we only have to modify one CI job if we want to change something, add a new strategy, etc.

This way we only have to modify one CI job if we want to change
something, add a new strategy, etc.
If it's not here, it breaks the CI even though the setting is
specified in the test file - this is because the environment
variable is accessed before the test setting is run, so it takes
precedence and causes an error (in CI).
Copy link

codecov bot commented May 8, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

❗ No coverage uploaded for pull request base (main@1abf702). Click here to learn what that means.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #558   +/-   ##
=======================================
  Coverage        ?   93.21%           
=======================================
  Files           ?      122           
  Lines           ?     5067           
  Branches        ?        0           
=======================================
  Hits            ?     4723           
  Misses          ?      344           
  Partials        ?        0           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@amstilp amstilp merged commit caefe37 into main May 8, 2024
9 checks passed
@amstilp amstilp deleted the maint/ci-combined-tests branch May 8, 2024 16:45
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.

1 participant