Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make dependency checker configurable #173

Open
jensens opened this issue Sep 5, 2023 · 3 comments
Open

Make dependency checker configurable #173

jensens opened this issue Sep 5, 2023 · 3 comments
Labels
04 type: enhancement making existing stuff better 14 prio: low nice to have it fixed 42 lvl: moderate fair project insight needed

Comments

@jensens
Copy link
Sponsor Member

jensens commented Sep 5, 2023

Like, for use outside of Zope world.

@jensens jensens added the 04 type: enhancement making existing stuff better label Sep 5, 2023
@gforcada
Copy link
Sponsor Member

🤔 do you want to disable it, so you have an option to not configure it, or rather that the options of dependecychecker to be configurable via plone/meta?

For the later that's the case I think, for the former, is it that you don't want it at all, fulll stop, or rather that it does not find the dependencies as expected?

@jensens
Copy link
Sponsor Member Author

jensens commented Sep 19, 2023

IIRC one can't get rid of the zope configs.

@gforcada gforcada added 14 prio: low nice to have it fixed 42 lvl: moderate fair project insight needed labels Oct 14, 2023
@gforcada
Copy link
Sponsor Member

Oh, I see, indeed.

We could add an new setting, plone_mappings (naming improvements welcome), set to True by default, that would allow one to set it to False and disable them.

That should be easy, contributions welcome 😉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
04 type: enhancement making existing stuff better 14 prio: low nice to have it fixed 42 lvl: moderate fair project insight needed
Projects
None yet
Development

No branches or pull requests

2 participants