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

Support linting of Zarf packages #359

Closed
gscallon opened this issue Jan 26, 2024 · 1 comment · Fixed by #436
Closed

Support linting of Zarf packages #359

gscallon opened this issue Jan 26, 2024 · 1 comment · Fixed by #436
Labels
enhancement New feature or request good first issue Good for newcomers

Comments

@gscallon
Copy link

gscallon commented Jan 26, 2024

Is your feature request related to a problem? Please describe

I am attempting to lint the Zarf package in a UDS task runner workflow. I am receiving the following error:

gavinscallon@Gavins-MacBook-Pro-2 uds-runner-pipeline-python-hello-world % uds zarf dev lint -f zarf.schema.json

 NOTE  Saving log file to
       /var/folders/37/qqg82c0x2hx43pczwp305f0r0000gn/T/zarf-2024-01-25-16-23-05-1549175322.log
     ERROR:  open zarf.schema.json: file does not exist

The schema file appears to get compiled with the Zarf binary when used with the standalone Zarf binary. The linting is successful if we run zarf dev lint.

Describe the solution you'd like

  • Given a Zarf.yaml file exists
  • When I go to run uds zarf dev lint
  • Then I am able to lint the Zarf package

Describe alternatives you've considered

(optional) A clear and concise description of any alternative solutions or features you've considered.

Additional context

Add any other context or screenshots about the feature request here.

@gscallon gscallon added the enhancement New feature or request label Jan 26, 2024
@decleaver decleaver removed their assignment Jan 29, 2024
@Racer159 Racer159 added the good first issue Good for newcomers label Feb 16, 2024
@Racer159
Copy link
Contributor

Planning to pick this one up - since the zarf schema is in the repo now just gonna pick that up (let me know if a different path is desired)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants