Skip to content

Remove upper bound on dependencies#66

Merged
lnxpy merged 1 commit intolnxpy:mainfrom
Viicos:relax-version
Aug 22, 2025
Merged

Remove upper bound on dependencies#66
lnxpy merged 1 commit intolnxpy:mainfrom
Viicos:relax-version

Conversation

@Viicos
Copy link
Copy Markdown
Contributor

@Viicos Viicos commented Jul 30, 2025

It is considered a bad practice to add upper bounds to dependencies for libraries (not to be confused with applications).

Further reading: https://iscinumpy.dev/post/bound-version-constraints/#pinning-the-python-version-is-special

If you are worrying about tests that could break, you can use a lock file.

@lnxpy
Copy link
Copy Markdown
Owner

lnxpy commented Aug 22, 2025

I agree with that, @Viicos. Thanks for mentioning and working on it! ;D
How did you find out about PyAction btw? (Just out of curiosity since you're also a member of Pydantic)

@lnxpy lnxpy merged commit ceecf15 into lnxpy:main Aug 22, 2025
7 checks passed
@Viicos
Copy link
Copy Markdown
Contributor Author

Viicos commented Aug 27, 2025

How did you find out about PyAction btw? (Just out of curiosity since you're also a member of Pydantic)

I think I stumbled upon it a while ago, and came back to it when exploring options to run complex actions. Can't remember exactly for what for.

@lnxpy
Copy link
Copy Markdown
Owner

lnxpy commented Aug 28, 2025

@Viicos cool. Thanks again.

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.

2 participants