Skip to content

Conversation

@nstarman
Copy link
Contributor

@nstarman nstarman commented Nov 14, 2024

Requires #217

@nstarman nstarman force-pushed the optional-dependencies branch 2 times, most recently from 39a056b to 0fb7657 Compare November 14, 2024 18:21
@wesselb
Copy link
Member

wesselb commented Nov 15, 2024

@nstarman That is unfortunate! I wasn’t aware that dependency groups are possible. I love it!

@nstarman
Copy link
Contributor Author

Would you be interested to use uv in the CI? https://docs.astral.sh/uv/guides/integration/github/#syncing-and-running
uv is not a build backends, so there's no change re hatch, its just the frontend commands. See https://github.com/GalacticDynamics/unxt for an example where hatch is the backend but uv is used for installing / running stuff.

@wesselb
Copy link
Member

wesselb commented Nov 15, 2024

@nstarman I think so! I haven’t used uv, but it looks great. Would you recommend uv?

@nstarman
Copy link
Contributor Author

Very much so!

@wesselb
Copy link
Member

wesselb commented Nov 15, 2024

Let’s do it then!!

@nstarman nstarman force-pushed the optional-dependencies branch 2 times, most recently from ef925ca to 65e4341 Compare November 18, 2024 04:12
@nstarman nstarman force-pushed the optional-dependencies branch 5 times, most recently from 5f7a869 to 446acff Compare September 30, 2025 18:13
@coveralls
Copy link

coveralls commented Sep 30, 2025

Pull Request Test Coverage Report for Build 19619133384

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 99.613%

Totals Coverage Status
Change from base Build 18974187178: 0.0%
Covered Lines: 1287
Relevant Lines: 1292

💛 - Coveralls

@nstarman nstarman force-pushed the optional-dependencies branch 7 times, most recently from 8917da8 to 8f387be Compare September 30, 2025 20:06
@nstarman nstarman mentioned this pull request Sep 30, 2025
@nstarman nstarman force-pushed the optional-dependencies branch 3 times, most recently from ec38565 to ff09f5e Compare November 23, 2025 23:26
Signed-off-by: nstarman <nstarman@users.noreply.github.com>
@nstarman nstarman force-pushed the optional-dependencies branch 3 times, most recently from b243a02 to 35a6714 Compare November 23, 2025 23:37
@nstarman nstarman force-pushed the optional-dependencies branch from 35a6714 to 7fb86d7 Compare November 23, 2025 23:41
Signed-off-by: nstarman <nstarman@users.noreply.github.com>
@nstarman nstarman force-pushed the optional-dependencies branch from 7fb86d7 to 43fc07e Compare November 23, 2025 23:44
@nstarman nstarman marked this pull request as ready for review November 23, 2025 23:49
@nstarman
Copy link
Contributor Author

@wesselb. Finally ready for review!

@wesselb
Copy link
Member

wesselb commented Nov 24, 2025

@nstarman Looks great!! Merging right away. :)

@wesselb wesselb merged commit ff1238d into beartype:master Nov 24, 2025
18 checks passed
@wesselb
Copy link
Member

wesselb commented Nov 24, 2025

@nstarman Hmm, the docs seem to fail to build now on main. Perhaps the GHA needs a pip install --upgrade pip as it doesn't seem to install the included dep group.

@nstarman nstarman deleted the optional-dependencies branch December 3, 2025 18:28
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.

3 participants