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

fix: Fix trunk.yaml lint error #268

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

martimlobao
Copy link

  • this file is causing my CI checks to fail because it fails yamllint

- this file is causing my CI checks to fail because it fails yamllint
@martimlobao
Copy link
Author

For context, here's an example: https://github.com/martimlobao/regybox/runs/33149046796

I only enabled the PR check from the Trunk app, the repo where the error is happening hasn't changed, and the file Trunk complains about is being downloaded from here.

@martimlobao
Copy link
Author

A more robust solution would be to have the linters not lint the contents of the .trunk directory itself, but I'm not sure how to do that and this works as an easy fix for now.

@martimlobao martimlobao changed the title Fix trunk.yaml lint error fix: Fix trunk.yaml lint error Nov 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

1 participant