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 metadata linting in PR checks #1

Open
patcon opened this issue Feb 7, 2021 · 1 comment
Open

Add metadata linting in PR checks #1

patcon opened this issue Feb 7, 2021 · 1 comment

Comments

@patcon
Copy link
Member

patcon commented Feb 7, 2021

Not for checking the whole config file, but just checking format of our metadata key, which is being used to keep track of:

  1. the repository the code comes from that's using the subdomain, and
  2. the maintainer(s) responsible for whatever lives at the subdomain

Could also use the linter to check for has admin control of the domain. I'm currently storing this in a TXT entry at the root level of each domain, as admin=foo.

https://github.com/23andMe/Yamale
https://github.com/nrkno/yaml-schema-validator-github-action

@patcon
Copy link
Member Author

patcon commented Mar 1, 2021

  • check that root domain has an admin=username TXT value
  • check that every A and CNAME record has:
    • a repository or url key
    • a maintainer key
  • check that TXT entries starting with 301 have an entry for ronny's app (and vice versa)
  • ensure all domain config files are *.yaml (NOT *.yml)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant