-
Notifications
You must be signed in to change notification settings - Fork 2
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
Created GH app for an Issue Manager, scripts for app, and deleted previous workflows. #9
Conversation
Hi @JamesAlfonse, I've just reviewed the shared chat log and will dive into the code nuances asap. You've done an incredible job thinking of these innovations and extensively building out a solution! Do I understand correctly that the chief goal here is adding items to a unified Projects board, superceeding the past outline script? Again, really stellar job going into the explicit details to get this one through 🚀 |
@JFWooten4 Thank you John!! That's correct, when this script runs (daily at midnight UTC) it should add new issues to the unified Project under a column named after each repository. I had the scripts tweaked slightly for more efficiency handling edge cases + added more documentation, so this should theoretically run more smoothly. Thanks for help with reviewing it! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks stellar James! Perhaps one thing we could chat about tonight is a preference org-wide for 2-space indents v. tabs. All around quite the comprehensive contribution, and most of my notes are simple procedural grounds. 🌌
Thank you for the in-depth review and feedback! Yes let's go over it tonight @JFWooten4 |
@JFWooten4 @tehchives Going to merge this pull request either tonight or tomorrow, I wanted to wait a little longer in case anything else came up. I adjusted the App settings to remove write permissions for Issues as I believe read-only should be sufficient. |
Read permissions for Issues was not sufficient lol. Adjusting that now + doing more testing to get this working. |
Used o1 preview to create GH app and generate scripts. Asked for edge cases and incorporated suggestions to deal with them. Added line-by-line documentation, looking for help on code review (especially with the logic on the edge cases).
https://chatgpt.com/share/6732de5a-492c-800c-bd03-a42113581c27