Periodic Labeler Exit Code 255 error on every run about periodic-labeler HOT 8 OPEN

Ok! I will, as soon as I can reproduce exactly the problem.
I tried now in my fork.. and recursivity works fine, but it doesn't in the PRs of my upstream, it might be related to forked branches, not sure, so as soon I clarify I open an issue to discuss about it ;)
Thanks Paul.

from periodic-labeler.

Apparently, the 2nd one from labeler example isn't supported:

#WORKS
# Add 'label1' to any changes within 'example' folder or any subfolders
label1:
  - example/**/*

#DOESN'T WORK
# Add 'label2' to any file changes within 'example2' folder
label2: example2/*

from periodic-labeler.

Yeah, I didn't test something/**/* matchers... Could you open a separate issue about it as it is a different bug?

from periodic-labeler.

/**/* support added in #8

from periodic-labeler.

Apparently, the 2nd one from labeler example isn't supported:

#WORKS
# Add 'label1' to any changes within 'example' folder or any subfolders
label1:
  - example/**/*

#DOESN'T WORK
# Add 'label2' to any file changes within 'example2' folder
label2: example2/*

Hi,

In my case it works only with the second example 🤷‍♂️

from periodic-labeler.

@srbarrios are you sure? I am personally using the first example in all @cloudalchemy repositories (example) and it works there. Also, I investigated codebase and the second example cannot possibly work in golang due to how the data structure is defined (https://github.com/paulfantom/periodic-labeler/blob/master/main.go#L93) - it expects a map of lists.

I am open to PRs fixing this issue, but personally I don't have time to add this right now.

from periodic-labeler.

Hi @paulfantom , here https://github.com/uyuni-project/uyuni/pull/1711/files I tried to reformat using the first example, as I thought the problem was to have the filter inline with the label, instead of having it in a new line. But then the matcher /**/* didn't catch any file, then I tried only with /* and it catched the files in the root folder but still the recursivity is not working for me.

TBH I didn't have time to play a lot with that GitHub Action. I might be still doing something wrong, looking into your commented example, it might work if I include both /**/* and /*.

I'll let you know when I have some minutes to play with it :)

Thank you!

from periodic-labeler.

Is there any news?

from periodic-labeler.