errose28 opened a new pull request, #87:
URL: https://github.com/apache/ozone-site/pull/87

   ## What changes were proposed in this pull request?
   
   Use [yamllint](https://github.com/adrienverge/yamllint) to to check yaml 
configuration files. This isn't a very important check IMO since most website 
changes will not be editing yaml files, and the `_category_.yml` files are 
pretty simple. However [yamllint is already in the GitHub actions Ubuntu 
runner](https://github.com/actions/runner-images/blob/main/images/ubuntu/Ubuntu2204-Readme.md)
 so it's low effort to add this check.
   
   I set the yaml linting rules based on what I usually follow but I don't have 
strong opinions on them as long as they are consistent.
   
   ## What is the link to the Apache Jira?
   
   HDDS-10351
   
   ## How was this patch tested?
   
   - Failing CI run with yamllint added but existing errors not enforced: 
https://github.com/errose28/ozone-site/actions/runs/8427283082/job/23077474962
   
   - CI on this PR should pass with the fixed yaml files.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to