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

Inconsistent behavior - dev mode overrides looseApplication=false in plugin parameter config but not property config #1818

Open
scottkurz opened this issue Jul 12, 2024 · 0 comments

Comments

@scottkurz
Copy link
Member

Though clearly dev mode is designed assuming the use of loose applications, it is still useful with loose application support disabled. See discussion: #1817

However, the intention at one point was to disable a looseApplication=false setting and force the user to run with the default looseApplication=true.

However, this override behavior was incompletely implemented. It was only implemented for plugin parameter config, but not properties.

Presumably this was an oversight at the time? There are other areas like the rework of the dev mode "skip" config where I believe we've tried to treat plugin & property config similarly.

Though one conceivable option would be to begin override the property behavior as well, like the parameter config, I think there is value in allowing this, as mentioned in the discussion above.

We could also simply stop overriding the parameter config.

Consider though a user that relies on loose app = false for some other use case, say some kind of custom Docker build. Perhaps that user has gotten used to also running dev mode.

Should we just leave things as-is then?

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

No branches or pull requests

1 participant