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

consistency in dashes Vs underscores (e.g. "env-vars") #11424

Open
smondet opened this issue Jan 31, 2025 · 2 comments · May be fixed by #11442
Open

consistency in dashes Vs underscores (e.g. "env-vars") #11424

smondet opened this issue Jan 31, 2025 · 2 comments · May be fixed by #11442
Labels
config-file Everything related to dune configuration file (workspace, project, dune) feature-request User wanted features

Comments

@smondet
Copy link

smondet commented Jan 31, 2025

See an example of rare use of dashes:
https://github.com/ocaml/dune/blob/main/src/dune_rules/dune_env.ml#L200

This can make search in the documentation not great.

The env-vars field could just have an env_vars alias and be backwards compatible.

@maiste maiste added feature-request User wanted features config-file Everything related to dune configuration file (workspace, project, dune) labels Feb 3, 2025
@maiste
Copy link
Collaborator

maiste commented Feb 3, 2025

It seems indeed inconsistent. @rgrinberg was there a reason and, if no, do you think we should do have them both living in dune?

@rgrinberg
Copy link
Member

There's no reason why it's this way. Adding an alias is fine here.

@maiste maiste linked a pull request Feb 4, 2025 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
config-file Everything related to dune configuration file (workspace, project, dune) feature-request User wanted features
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants