-
-
Notifications
You must be signed in to change notification settings - Fork 7
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
Feature/add gitignore, pr template - DO NOT MERGE #38
Conversation
All contributors have signed the CLA ✍️ ✅ |
Trying to figure out how ben's comment did generate the json entry in signatures. I may take a stab at it later. |
Seems to be some type of cla-bot issue, could be permissions? Not automating/picking up the comments from the signed CLA? |
I'm trying to figure out the CLA issue so I can push these into the repo. Could you try adding the "I have read the CLA Document and I hereby sign the CLA" again as a comment in this Issue #38 ? Also, I wasn't aware of github "runners". How cool! |
Oh - hey - is it possible you're adding a comment directly to the gist, instead of here as a comment in the issue? I think it has to be the latter. |
This partially addresses #41 — can be closed once non-PR issues also get a template (or if it's determined not to be the right thing for the repo 🙂) |
Hi there - Just wanted to remind you to try the CLA process again (see comments above). Thank you! |
I have read the CLA Document and I hereby sign the CLA |
Looks like there may have been a permissions issue. https://github.com/jamespilgrim/PiTrac/actions/runs/12637411892/job/35211538806 I also noticed you're requiring signed commits too. I'll configure that as well. |
5a6087c
to
11a4461
Compare
11a4461
to
ce312b3
Compare
Closing due to additional changes being introduced. |
Testing feature to main PR. Are we using github runners? Example
runs-on: [ubuntu-latest]