Skip to content

Update dagster-apprise lockfile#300

Merged
cmpadden merged 3 commits into
mainfrom
update-dagster-apprise-lockfile
May 22, 2026
Merged

Update dagster-apprise lockfile#300
cmpadden merged 3 commits into
mainfrom
update-dagster-apprise-lockfile

Conversation

@cmpadden

Copy link
Copy Markdown
Contributor

Summary

  • refresh dagster-apprise uv.lock with latest compatible dependency versions

Testing

  • uv lock --upgrade in libraries/dagster-apprise

@cmpadden

Copy link
Copy Markdown
Contributor Author

Pulled the failing check logs with gh. The failure was from the upgraded Ruff formatter reporting files that needed formatting. Ran uv run ruff format in libraries/dagster-apprise, committed the formatting-only fix, and pushed it to this branch.

@cmpadden

Copy link
Copy Markdown
Contributor Author

Pulled the new failing check logs with gh after the formatting fix. The updated Apprise dependency now accepts webex://webhook_url, so the invalid URL expectation was stale. Removed that URL from the invalid list and verified locally with uv run pytest dagster_apprise_tests/test_comprehensive_urls.py::TestComprehensiveAppriseUrls::test_invalid_urls -q, then pushed the fix.

@cmpadden cmpadden merged commit ed217b7 into main May 22, 2026
7 checks passed
@cmpadden cmpadden deleted the update-dagster-apprise-lockfile branch May 22, 2026 18:27
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