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

Remove useless function step_progress_update #2059

Merged
merged 1 commit into from
Jul 30, 2024

Conversation

erikbern
Copy link
Contributor

Seems a bit silly for this to be its own function

Copy link

@modal-pr-review-automation modal-pr-review-automation bot left a comment

Choose a reason for hiding this comment

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

Auto-approved 👍. This diff qualified for automatic approval and doesn't need follow up review.

@erikbern erikbern force-pushed the erikbern/remove-step-progress-update branch from fada8e3 to 5f53a91 Compare July 29, 2024 23:52
@erikbern erikbern force-pushed the erikbern/remove-step-progress-update branch from 5f53a91 to 93e6b53 Compare July 30, 2024 00:00
@erikbern erikbern merged commit d52f776 into main Jul 30, 2024
20 checks passed
@erikbern erikbern deleted the erikbern/remove-step-progress-update branch July 30, 2024 00:10
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