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

should YAML files be .yml or .yaml? #624

Closed
jbathegit opened this issue Oct 9, 2024 · 4 comments · Fixed by #634
Closed

should YAML files be .yml or .yaml? #624

jbathegit opened this issue Oct 9, 2024 · 4 comments · Fixed by #634

Comments

@jbathegit
Copy link
Collaborator

@edwardhartnett @AlexanderRichert-NOAA @Hang-Lei-NOAA

I've noticed that all of the YAML filenames in all of the NCEPLIBS .github/workflows subdirectories seem to use .yml as a suffix rather than .yaml. Is there any particular reason for this?

I'm asking b/c the latter seems to be more the preferred industry standard, at least according to https://yaml.org/faq.html and https://stackoverflow.com/questions/22268952/what-is-the-difference-between-yaml-and-yml-extension. In fact, the latter is already used within the spack/spack.yaml file within the NCEPLIBS-bufr repository, so there's also some inconsistency in that regard if we continue to use .yml for the files within .github/workflows.

Again, not a big deal, but just curious if there's any particular reason for this, and if not then would it be worthwhile to try and move towards what seems to be the preferred industry standard here?

@Hang-Lei-NOAA
Copy link
Contributor

Hang-Lei-NOAA commented Oct 9, 2024 via email

@jbathegit
Copy link
Collaborator Author

@Hang-Lei-NOAA @edwardhartnett @AlexanderRichert-NOAA

So we seem to agree that it would be good to be consistent across all of the NCEPLIBS, but do we have an agreement about which extension to gravitate towards? It seems to me that the .yaml extension may be the preferred industry standard, but I could be wrong about that(?)

@edwardhartnett
Copy link
Contributor

Only one way to settle this: https://www.youtube.com/watch?v=Chv9TPtaX-E

@jbathegit
Copy link
Collaborator Author

Only one way to settle this: https://www.youtube.com/watch?v=Chv9TPtaX-E

OK that seems a bit harsh :-))

Does anyone have a problem if I try making this change to use .yaml throughout NCEPLIBS-bufr? Then, assuming it works w/o any problems, others can likewise switch over the other NCEPLIBS at some later date so they're all consistent?

@jbathegit jbathegit changed the title should YAML files by .yml or .yaml? should YAML files be .yml or .yaml? Dec 6, 2024
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

Successfully merging a pull request may close this issue.

3 participants