staabm / phpstan-todo-by

Todo comments with expiration
https://staabm.github.io/2023/12/17/phpstan-todo-by-published.html
MIT License
172 stars 4 forks source link

Respect configurable vendor folder location #85

Closed DanielSiepmann closed 6 months ago

DanielSiepmann commented 6 months ago

This can be seem as follow up to https://github.com/staabm/phpstan-todo-by/pull/54#discussion_r1440652936

We now have a project where best practices define to move the vendor folder. This is pretty common for TYPO3 Extension. You can find the repository here: https://github.com/FriendsOfTYPO3/tea it serves as a best practice for the community and we want to add the to do extension.

But that doesn't work due to the different vendor location. It therefore would be cool if the extension would read the location and that way support the actual path.

staabm commented 6 months ago

Using a non-standard vendor folder brings a lot of problems with different tools.

I would warn you to recommend this to a whole community.

I am also not willing to build a PR for such a way of doing stuff as I really don't like this idea.

That said, in case you provide a automated test and the fix does not require a lot of changes, I am open for a contribution

DanielSiepmann commented 6 months ago

I'm not aware of any issues. I also do not use that, but the community decided it is best practice.

Thanks for your honesty. I fully understand that and believe in the right as maintainer to decide so.

I'll use your refusal to discuss the vendor location with the best practices team. I'll close the issue for now. I might re-open it if we decide to contribute with testing.

It sounds like you are aware of issues, can you share some for reference?

staabm commented 6 months ago

from the top of my head:

Ocramius commented 6 months ago

has a special place in his heart for people who deviate from the eco-system wide default ;-)

It's really just your same reasoning: tons and tons of complexity. Imagine moving the .git directory and expecting all git-related tools to work 😱

github-actions[bot] commented 5 months ago

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.