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

Improve local testing for all RDBMS #75

Merged
merged 15 commits into from
Nov 5, 2024
Merged

Improve local testing for all RDBMS #75

merged 15 commits into from
Nov 5, 2024

Conversation

arogachev
Copy link
Contributor

Q A
Is bugfix?
New feature?
Breaks BC?

@arogachev arogachev self-assigned this Oct 18, 2024
@arogachev arogachev added the type:task Task label Oct 18, 2024
@arogachev arogachev added this to the 1.0 milestone Oct 18, 2024
Copy link

codecov bot commented Oct 26, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 76.33%. Comparing base (174652b) to head (364f114).
Report is 23 commits behind head on master.

Additional details and impacted files
@@             Coverage Diff              @@
##             master      #75      +/-   ##
============================================
- Coverage     77.55%   76.33%   -1.22%     
- Complexity      105      108       +3     
============================================
  Files            22       22              
  Lines           294      300       +6     
============================================
+ Hits            228      229       +1     
- Misses           66       71       +5     

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

@arogachev arogachev marked this pull request as ready for review October 28, 2024 16:03
@arogachev arogachev requested a review from a team October 28, 2024 16:03
@arogachev arogachev added the status:code review The pull request needs review. label Oct 28, 2024
Makefile Outdated Show resolved Hide resolved
Dockerfile Outdated Show resolved Hide resolved
docker-compose.yml Show resolved Hide resolved
arogachev and others added 2 commits October 30, 2024 17:55
Co-authored-by: Alexander Makarov <[email protected]>
This reverts commit ded8b11.
@samdark
Copy link
Member

samdark commented Nov 4, 2024

It's good at this stage, I think. Let's merge it.

@arogachev
Copy link
Contributor Author

arogachev commented Nov 5, 2024

@samdark I've fixed Makefile as well.

@arogachev arogachev merged commit 08ed4ed into master Nov 5, 2024
53 checks passed
@arogachev arogachev deleted the docker branch November 5, 2024 18:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status:code review The pull request needs review. type:task Task
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants