https://pantsbuild.org/ logo
p

polite-garden-50641

05/20/2020, 12:53 AM
looking at the issues caused by the GH action, working on a fix.
h

hundreds-father-404

05/20/2020, 12:54 AM
Oh, hm, what issue? It worked for Benjy’s PR when I added a label
p

polite-garden-50641

05/20/2020, 12:55 AM
it's busted... not sure. looking into it.
Screenshot 2020-05-19 17.54.49.png
h

happy-kitchen-89482

05/20/2020, 12:56 AM
Thanks
##[error]Can't find 'action.yml', 'action.yaml' or 'Dockerfile' under '/home/runner/work/pants/pants'. Did you forget to run actions/checkout before running your local action?
h

hundreds-father-404

05/20/2020, 12:57 AM
Will that new PR use the updated config file, or it only uses that after a merge? edit: looks like it uses the updated file. That’s helpful
p

polite-garden-50641

05/20/2020, 1:02 AM
ok. looks like someone with more permissions than myself needs to add a GH token. instructions here: https://help.github.com/en/actions/configuring-and-managing-workflows/authenticating-with-the-github_token
hmm... this should work event without... I am looking into it.
h

hundreds-father-404

05/20/2020, 1:07 AM
Is that saying that the token will only be populated when the author has write access to pantsbuild/pants? I don’t think so, because it still fails on Benjy’s PR, but unclear
p

polite-garden-50641

05/20/2020, 1:13 AM
I am testing stuff on my own repo... I'll have answers soon.
❤️ 1
h

hundreds-father-404

05/20/2020, 1:14 AM
Thanks! I pointed out to Benjy that we’re allowed to merge on red, so master isn’t quite “broken”, but certainly appreciated that you’re working on this.
p

polite-garden-50641

05/20/2020, 1:17 AM
I think the issues is because I submitted the PR.
I have the same file in my own repo and it works just fine.
@hundreds-father-404 can you copy my changes and submit the PR ?
Screen Shot 2020-05-19 at 6.18.30 PM.png
h

hundreds-father-404

05/20/2020, 1:18 AM
Huh. Do you think it’s enough for me to do you PR that edits things, or we need to delete the file and start over?
p

polite-garden-50641

05/20/2020, 1:20 AM
I think you can just edit the existing changes...
basically submit my diff
h

hundreds-father-404

05/20/2020, 1:20 AM
Open a new PR with that diff, right?
p

polite-garden-50641

05/20/2020, 1:21 AM
yes
👍 1
h

hundreds-father-404

05/20/2020, 1:21 AM
Will do. Just finished the release
p

polite-garden-50641

05/20/2020, 1:22 AM
this action works fine on my repo (well... except for the fact it generates multiple comments... but we can address that later)
I can probably improve that... will do that soon.
❤️ 1
It makes sense that it doesn't let my PR access the token... since I could easily point to an action that I write and upload the token to my server....
h

hundreds-father-404

05/20/2020, 1:24 AM
It would probably be most ideal if we could turn it off. We don’t allow first-time contributors to add labels, so it’s a frustrating UX to get yelled at that they need to do this and they can’t. (I’ll add a warning to the contributors guide at https://pants.readme.io/docs/contribution-flow)
p

polite-garden-50641

05/20/2020, 1:25 AM
so what is going on makes total sense.
h

hundreds-father-404

05/20/2020, 1:25 AM
p

polite-garden-50641

05/20/2020, 1:26 AM
ok. you should delete this yaml, merge it. and then submit it yourself. if it doesn't work, we might need someone with more permissions...
h

hundreds-father-404

05/20/2020, 1:26 AM
Possibly Benjy, John, or Stu. They’re project owners, whereas I’m a maintainer
p

polite-garden-50641

05/20/2020, 1:28 AM
ok. I think this needs more work. I'll revert my original PR for now.
I'll try to see if there is a more user friendly way to handle this.
I think this should somehow integrate with the GH checks instead of adding a request changes comment. also, I can probably add some logic to allow contributors to specify labels somehow.
I think for now you and other committers can manually enforce labeling.
h

hundreds-father-404

05/20/2020, 1:31 AM
I can probably add some logic to allow contributors to specify labels somehow
I think that’s a GitHub permissions thing. Also, I’m not certain we want to enforce that the contributor makes that classification
Oh, I had already put up https://github.com/pantsbuild/pants/pull/9832 a little before and pinged people for reviews
w

witty-crayon-22786

05/20/2020, 1:50 AM
🚢
❤️ 1