Skip to content

Little fix - #9

Merged
dmanuel64 merged 5 commits into
dmanuel64:mainfrom
lerlyi:little_fix
Aug 8, 2025
Merged

Little fix#9
dmanuel64 merged 5 commits into
dmanuel64:mainfrom
lerlyi:little_fix

Conversation

@lerlyi

@lerlyi lerlyi commented Aug 6, 2025

Copy link
Copy Markdown
Contributor

Adding a dependency without which docker-compose will not start

lerlyi added 2 commits August 6, 2025 21:15
Add missing dependency 'deprecated' to requirements.txt
Add missing dependency 'deprecated' to requirements.txt

@dmanuel64 dmanuel64 left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Thanks for pointing this out. Since the Dockerfile installs the package directly through pyproject.toml, the dependencies list under the [project] section and the all list under the [project.optional-dependencies] need to be appended with the following line:

"Deprecated>=1.2.18"

Comment thread docs/requirements.txt Outdated
mkdocs-section-index>=0.3.9
mkdocstrings[python]>=0.26.0 No newline at end of file
mkdocstrings[python]>=0.26.0
deprecated

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

This requirements.txt file is somewhat misleading. This file is actually used by mkdocs to serve documentation. So all the dependencies in this file, have nothing to do with the package and there shouldn't be anything here that needs to be changed. The underlying issue is that deprecated isn't in pyproject.toml.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Thank you, I am a little bit stupid, but I really want to launch your project

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

No worries, I'll probably be updating it in the upcoming days with updated docs, unit tests, an easier way to launch it from Docker.

Comment thread pyproject.toml

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

It looks like tree-sitter introduced a breaking change that is now causing the package to fail. I'll have to look into upgrading tree-sitter to it's newest version at a later time. In the meantime, can you please pin the version so it will run?

"tree-sitter==0.23.2"

@dmanuel64 dmanuel64 added the bug Something isn't working label Aug 8, 2025
@dmanuel64
dmanuel64 merged commit d6bb776 into dmanuel64:main Aug 8, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants