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

Run YAML validity/schema linters before snapcraft #58

Open
abitrolly opened this issue Sep 24, 2024 · 3 comments
Open

Run YAML validity/schema linters before snapcraft #58

abitrolly opened this issue Sep 24, 2024 · 3 comments

Comments

@abitrolly
Copy link

abitrolly commented Sep 24, 2024

To avoid strange errors, because of typos and general inability to remember documentation, it would be nice to run YAML structure linters before snapcraft.

Docs mention that snapcraft runs embedded linters https://snapcraft.io/docs/linters but they don't catch some low level errors like absent base: key (#57 (comment)).

EDIT: And troubleshooting such silly mistakes can steal a lot of time.

@abitrolly
Copy link
Author

This issue in #57 was caused by older version of snapcraft producing parts/hello-part/state/pull that newer version could not load.

The best architecture is to have a mechanism to version build environment and migrate it.

@tigarmo
Copy link
Collaborator

tigarmo commented Sep 26, 2024

There are two issues here:

  1. The lack of base is, and should be, handled by Snapcraft itself. The problem is the default run of the image doesn't show Snapcraft's output easily, which is the more pressing issue I think.
  2. That second one is also a bug in Snapcraft, or craft-parts. I don't think it's a bug here.

@abitrolly
Copy link
Author

@tigarmo I've got an issue fatigue today. Maybe mods on the payroll can move or reticket it to the relevant repos?

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

No branches or pull requests

2 participants