Closed mollybsmith-noaa closed 3 months ago
@mollybsmith-noaa and @fisherhucar There was some discussion of this topic in Slack on .yml vs. .yaml. Minna and I thought the files were to be renamed .yaml and not .yml. Please see the discussion in the METplus Analysis channel. I'll go tag you both there now.
Actually, I see that @mollybsmith-noaa added some comments, which I hadn't seen previously. It sounds like we're waiting to hear back from others before deciding on anything.
@fisherhucar No worries. I won't be at the meeting due to LEAD training, but I'm sure you all have will cover it well. I'll check in on the notes to see what I missed.
Pull request to change all .yaml files in METdataio to use .yml, ensuring uniformity.
Pull Request Testing
[ ] Describe testing already performed for these changes:
[ ] Have verified that METdbload still works but I don't really have the capacity to test METreadnc or METreformat
[ ] Recommend testing for the reviewer(s) to perform, including the location of input datasets, and any additional instructions:
[ ] If possible, test that METreadnc and METreformat are working as expected.
[ ] Do these changes include sufficient documentation updates, ensuring that no errors or warnings exist in the build of the documentation? -- Change to text of documentation, but no change to documentation structure itself.
[ ] Do these changes include sufficient testing updates? Yes -- Some test files have extensions changed from .yaml to .yml. If tests pass this should be fine.
[ ] Will this PR result in changes to the test suite? Yes, see above.
[ ] Do these changes introduce new SonarQube findings? No If yes, please describe:
[ ] Please complete this pull request review by 5/3/2024.
Pull Request Checklist
See the METplus Workflow for details.