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 passing in config file + add support for using globs #55

Closed
wants to merge 7 commits into from
Closed

Fix passing in config file + add support for using globs #55

wants to merge 7 commits into from

Conversation

prince-chrismc
Copy link

👋 Hi there, I was working on conan-io/conan-center-index#13274

And I needed a problem matcher for yamllint and discovered your action! (thank btw) sadly I had two small snags

  • the logic around the config file was flipped
  • I could not pass in a glob expression

This PR adds support for both of these but theres not really any testing put in place, but hopefully we could see that get added 🤞

My job gives me 20% for doing open source work, so if you need a collaborator to help review PRs I'd love triage issues 😉

@github-actions
Copy link

There hasn't been any activity on this pull request recently. This pull request has been automatically marked as stale because of that and will be closed if no further activity occurs within 7 days. Thank you for your contributions.

@github-actions github-actions bot added the stale There has not been activity on this issue or PR for quite some time. label Nov 21, 2022
@prince-chrismc prince-chrismc closed this by deleting the head repository Nov 21, 2022
@github-actions github-actions bot locked and limited conversation to collaborators Nov 23, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
stale There has not been activity on this issue or PR for quite some time.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant