Skip to content

fix: Unexpected input(s) 'predicate-quantifier' #226

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

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

Conversation

petermetz
Copy link
Contributor

Declared the input parameter in the action.yml file so GitHub should
no longer complain about invalid input when we use the new feature.

Fixes #225

Signed-off-by: Peter Somogyvari [email protected]

Declared the input parameter in the action.yml file so GitHub should
no longer complain about invalid input when we use the new feature.

Fixes dorny#225

Signed-off-by: Peter Somogyvari <[email protected]>
@Joshua-Douglas
Copy link

Thanks for fixing! I came here to do the same after running into linting errors from rhysd/actionlint.

@prathamVaidya
Copy link

@dorny I don't like the redness of the linting errors. Please merge this PR.

@damageboy
Copy link

+1

Interested in seeing this deployed too... :)

@watanabeyu
Copy link

+1

eagerly awaiting...

@mathieu-chiavassa
Copy link

Amazing, thank you so much !! Will it be possible to deploy these changes any time soon ? 🙏

@tyler-adams-kr
Copy link

+1 would love to see this released @petermetz @dorny

@momentmaker
Copy link

same - let's merge this so we can get rid of the annotation :)

@cebarobot
Copy link

+1. When will this PR merged into master? 😣

@ahsankarimbhai
Copy link

+1 Please merge this PR

@jcadavez
Copy link

When will this PR be merged

@JosiahSiegel
Copy link

maybe it's time to fork this action to add this change

@sarangard
Copy link

+1 Please review and merge this PR 🙏

@robbie-anam
Copy link

+1 eagerly awaiting a merge!

@ParGG
Copy link

ParGG commented Aug 21, 2024

+1 🙏

@hspitzley-czi
Copy link

hspitzley-czi commented Aug 23, 2024

Would love for this to be merged. cc @dorny

@randallt21
Copy link

Can we please get this merged. cc @dorny

@galexeyev
Copy link

@dorny Can we please get this merged?

@NeerajLaljiTHG
Copy link

@dorny can we get this merged please :)

@trombini77
Copy link

Please is possible to merge this fix to stop the false warning in the execution with predicate-quantifier arg?

@dswaffordcw
Copy link

This PR appears to be abandoned. I just put up a replacement under #258

@yanetrodriguez25
Copy link

@dorny Can we merge this PR please? thanks!!

@sasa-fajkovic
Copy link

@dorny Can we add more approves if you don't have time?

@sasa-fajkovic
Copy link

@JJ @jsoref @billyvg Maybe you folks can press the button to resolve this small annoyance :)

@jsoref
Copy link
Contributor

jsoref commented Apr 15, 2025

@sasa-fajkovic: this repository is not in an org, it's in @dorny's personal account, which generally means that only the owner can do releases.

Notes:

  1. I don't use this action (or really any action in this category)
  2. I'm not endorsing this action (or any other similar action)
  3. There's no requirement that you use the original version of an action, GitHub doesn't give any particular preference to any repository in a fork network when running actions.
  4. Fwiw, the PR itself seems fine.
  5. If I wanted to use this action, I'd just use uses: petermetz/paths-filter@5ee2f5d4cf5d7bdd998a314a42da307e2ae1639d
  6. Afaict, Michal is still alive (as of April 9) but has not publicly touched GitHub since de90cc6.

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

Successfully merging this pull request may close these issues.

fix: Unexpected input(s) 'predicate-quantifier'