Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Add Flow.ato_deployment, Flow.afrom_source, and Runner.afrom_storage #16897

Merged
merged 4 commits into from
Jan 30, 2025
Merged

Add Flow.ato_deployment, Flow.afrom_source, and Runner.afrom_storage #16897

merged 4 commits into from
Jan 30, 2025

Conversation

desertaxle
Copy link
Member

Introduces new explicitly async methods to use async_dispatch instead of sync_compatible.

Related to #15008

@codspeed-hq CodSpeed HQ
Copy link

codspeed-hq bot commented Jan 29, 2025 •
edited
Loading

CodSpeed Performance Report

Merging #16897 will not alter performance

Comparing remove-sync-compatible-from-flows (8fb1ead) with main (180185e)

Summary

✅ 2 untouched benchmarks

@desertaxle desertaxle marked this pull request as ready for review January 29, 2025 22:16
zzstoatzz
zzstoatzz approved these changes Jan 29, 2025
View reviewed changes
Copy link
Collaborator

@zzstoatzz zzstoatzz left a comment

Choose a reason for hiding this comment

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

🫡

@desertaxle desertaxle merged commit 38700d8 into main Jan 30, 2025
49 checks passed
@desertaxle desertaxle deleted the remove-sync-compatible-from-flows branch January 30, 2025 15:12
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@zzstoatzz zzstoatzz zzstoatzz approved these changes

@cicdw cicdw Awaiting requested review from cicdw cicdw is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@desertaxle @zzstoatzz