Charles Nicholson

Results 101 comments of Charles Nicholson

Hi @BagToad - The documentation says ``` -d, --delete-branch Delete the local and remote branch after merge ``` To me it feels like it would be odd for that to...

Also, thanks a lot for picking this one up! However it ends up being resolved, it's exciting to know that I won't be able to accidentally bury remote branches anymore...

Oh also, I think it closes the PR as well- I forgot to mention that in my original thread bump!

My personal preference is to view `gh pr merge -d 1234` as "Please try to merge this PR, and if it succeeds, delete both the local and remote branches and...

Thanks for looking into it! It still seems to be happening, unfortunately. I threw together a minimal reproduction in a public repo that you can clone and investigate, if you...

Happy to! I set up a manual CI trigger for it, you can see it happening on `ubuntu-latest` here: https://github.com/charlesnicholson/deptry-pep420-repro/actions/runs/9810619739/job/27091137978

Hmm, I'm still seeing the error in my little repro testbed on v0.19.1: The failed action is here: https://github.com/charlesnicholson/deptry-pep420-repro/actions/runs/10343972148/job/28628860626 and the repro, updated to use `--experimental-namespace-package` is here: https://github.com/charlesnicholson/deptry-pep420-repro (Is...

As it is, this PR alone inflates the binary size by a significant amount and I'm not sure what the benefits are. You have many PRs open and it's hard...

I do like your idea of making npf_parse_format_spec more user-friendly; I use it as a helper function in another project of mine. I'm not sure that I like the idea...

This PR has no changes to the codebase; is that intentional?