search

pulsar-edit / ppm

Pulsar Package Manager
MIT License
35 stars 13 forks source link

fix: Don't assume `master` when checking git packages for upgrades #56

Closed savetheclocktower closed 1 year ago

savetheclocktower commented 1 year ago

Fixes #55.

It took me ages to figure out how to replicate the test-git-repo.git fixture, but i managed to create one from scratch that uses main instead of master. All existing tests pass.

savetheclocktower commented 1 year ago

I think there may be places elsewhere in ppm that expect master in a hard-coded way, but I think we all want to move away from that, and this is as good a start on that as any.

The good news there is that I didn't find any others. The only implicit assumption of master that remains is in the code that converts a TextMate bundle on GitHub to an Atom package, and I can't imagine anyone is writing new TextMate bundles these days that would have a main default branch.