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.

Sign up for GitHub

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

Jump to bottom

Use the mvnd. prefix for all property names #221

Closed
gnodet opened this issue Nov 14, 2020 · 1 comment
Closed

Use the mvnd. prefix for all property names #221

gnodet opened this issue Nov 14, 2020 · 1 comment
Milestone
0.1.0

Comments

@gnodet
Copy link
Contributor

gnodet commented Nov 14, 2020

The environment properties are not always named in a consistent way.
We should make them more coherent.
One solution would be to have all properties beginning with mvnd. instead of daemon.
Also we have a few properties that use . to separate words instead of using a capital such as mvnd.daemon.storage or daemon.core.extensions.
@ppalaga
Copy link
Contributor

ppalaga commented Nov 15, 2020

+1 for mvnd.

daemon. does not sound specific enough. There are many daemons there in the world. mvndaemon. would work though.

gnodet added a commit to gnodet/mvnd that referenced this issue Nov 16, 2020
@gnodet
Clean the names of properties, fixes apache#221
ddde77c
gnodet added a commit to gnodet/mvnd that referenced this issue Nov 16, 2020
@gnodet
Clean the names of properties, fixes apache#221
bcf23d1
@gnodet gnodet closed this as completed in 668d4e4 Nov 16, 2020
@ppalaga ppalaga added this to the 0.0.13 milestone Nov 17, 2020
@ppalaga ppalaga changed the title Clean the names of properties Use the mvnd. prefix for all property names Nov 17, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
0.1.0
Development

No branches or pull requests
2 participants
@gnodet @ppalaga