Closed ecshreve closed 5 months ago
I did some research into this problem here: https://github.com/amonks/run/issues/94#issuecomment-2028923815
I think this script is useful, but, based on that research, I think it does a bit too much.
Plus I have some nits :)
FWIW: I'm using the #needs-thought label on issues to indicate that I'm unlikely to accept a PR without a lot of discussion. A first-pass PR is a great way to get that discussion started! But be warned: I will ask for changes without remorse.
Some people feel bad when they write code that isn't accepted, for labor-efficiency reasons. For those people, I'd recommend starting the discussion with a comment instead.
But I don't want to discourage you: if you're cool with rejection, keep throwing up PRs! It's definitely the most fun way to explore a design space, even if it isn't the most labor efficient. I'd do the same thing.
That is to say: thanks for kicking off this discussion! I'd love to work with you to get something merged, and I hope you aren't put off by all my feedback and opinions.
@amonks Thanks so much for taking the time to look at this, definitely not put off by the feedback at all, if anything I'm super appreciative and sorry if I made extra thinking time for you unnecessarily.
I'm trying to find a better workflow on my end where I can open a PR on a branch and work on the PR incrementally with the changes in the branch. If I open a PR in "draft" form is that enough to indicate "i don't expect anyone to review this, but this is something i'm looking at"? Or would it be better for me to keep things local and iterate on ideas in the issue discussion early on? Or could we enable PR labels in the repo and I could plop a "Not Ready for Review" label on WIP PRs? Interested in your thoughts on this.
I'm going to go ahead and close this PR and continue working on the change addressing #94 locally taking this PR feedback and your Issue comments into account. 🎉
sure; let's say from now on "draft" means "don't look at this unless you really extra feel like it"
(I don't have a lot going on right now, so I might, in practice, feel like looking at drafts, but that's me actively doing something know I don't need to, so you're not allowed to feel bad about creating extra work)
@ecshreve
tell me if I'm off base, but I think there might be a dynamic here where,
if so, the solution is: let's keep doing what we enjoy, and, as friends, choose not to feel bad about it :)
perfect, sounds like a plan to me!
first pass at https://github.com/amonks/run/issues/94
did a little testing with this github actions workflow, but not sure the right way to get it in here