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.

Sign up for GitHub

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

Jump to bottom

fix: wrap migrations from AsyncMigrations in Arc #174

Merged
merged 1 commit into from
Jul 23, 2024
Merged

fix: wrap migrations from AsyncMigrations in Arc #174

merged 1 commit into from
Jul 23, 2024

Conversation

cljoly
Copy link
Owner

@cljoly cljoly commented Jul 22, 2024 •
edited
Loading

This avoids some needless cloning. This change should be API compatible.

This was written by @fluffysquirrels, I have just extracted the code from #171

@cljoly cljoly added the enhancement New feature or request label Jul 22, 2024
@cljoly cljoly added this to the Version 1.3.0 milestone Jul 22, 2024
@cljoly
fix: wrap migrations from AsyncMigrations in Arc
abe0e70 
This avoids some needless cloning. This change should be API compatible.

Co-authored-by: Alex Helfet <alex.helfet@gmail.com>
@coveralls
Copy link

coveralls commented Jul 22, 2024 •
edited
Loading

Coverage Status

coverage: 95.224%. remained the same
when pulling abe0e70 on arc-async
into 63b6374 on master.

@cljoly cljoly added the performance Performance issues and improvement opportunities label Jul 22, 2024
@cljoly cljoly merged commit 56409a3 into master Jul 23, 2024
32 checks passed
@cljoly cljoly deleted the arc-async branch July 23, 2024 18:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
enhancement New feature or request performance Performance issues and improvement opportunities
Projects
None yet
Milestone
Version 1.3.0
Development

Successfully merging this pull request may close these issues.
2 participants
@cljoly @coveralls