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

Add CI/CD workflow to validate YAML files #40

Merged
merged 3 commits into from
Apr 25, 2024

Conversation

dthaler
Copy link
Collaborator

@dthaler dthaler commented Apr 23, 2024

  • Add CI/CD workflow to validate YAML files
  • Update YAML files to resolve yamllint-reported issues

The intent is that this should run on a pull request and be required to pass before the pull request can be merged.

Unlike the workflow in the GEDCOM and GEDCOM.io repositories, this workflow is configured to ignore trailing spaces since the generated YAML files have lots of trailing spaces (e.g., blank lines between paragraphs in the spec are converted to lines indented with four spaces) so the workflow would fail if not disabled.

Addressed part of #37

Signed-off-by: Dave Thaler <[email protected]>
@dthaler dthaler merged commit a3a3a4c into FamilySearch:main Apr 25, 2024
1 check passed
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 this pull request may close these issues.

2 participants