Skip to content

MAINT: remove cruft in dependabot.yml - #139

Open
tylerjereddy wants to merge 1 commit into
lanl:mainfrom
tylerjereddy:treddy_issue_133
Open

tylerjereddy wants to merge 1 commit into
lanl:mainfrom
tylerjereddy:treddy_issue_133

Conversation

@tylerjereddy

Copy link
Copy Markdown
Collaborator
  • Fixes MAINT: remove/update cruft in dependabot.yml #133.

  • We removed the .github/constraints path in MNT, CI: Update pyproject.toml to include [dependency-groups] #127, when we switched to dependency groups for managing dependencies, rather than using requirements files.

  • There's no point in having dependabot scheduled to update versions of dependencies stored at a path that no longer exists. I haven't found evidence that dependabot is able to properly handle auto-update of pyproject.toml dependency groups, so for now I propose just removing the associated config code. If a reviewer can find evidence that dependabot can handle such updates, please direct me to the approach/evidence and I'll adjust accordingly.


AI usage disclosure: no AI used here

* Fixes lanlgh-133.

* We removed the `.github/constraints` path in lanlgh-127, when
we switched to dependency groups for managing dependencies, rather
than using requirements files.

* There's no point in having dependabot scheduled to update
versions of dependencies stored at a path that no longer exists.
I haven't found evidence that dependabot is able to properly
handle auto-update of `pyproject.toml` dependency groups, so
for now I propose just removing the associated config code.
If a reviewer can find evidence that dependabot can handle
such updates, please direct me to the approach/evidence and
I'll adjust accordingly.
@tylerjereddy tylerjereddy added this to the 0.3.0 milestone Oct 21, 2026
@tylerjereddy

Copy link
Copy Markdown
Collaborator Author

We expect the CI to fail until gh-117 is resolved. I've opened gh-138 to propose xfailing the related Linux failures because that has been plaguing us for months now.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

MAINT: remove/update cruft in dependabot.yml

1 participant