User-created branches in the monorepo are often misused

I just had to delete these branches because I could not checkout llvm on Windows

  • users/minglotus-6/spr/main.nfcprecommit-test-case-to-show-function-summary-and-global-values-when-a-function-has-instructions-annotated-with-vtable-profiles-and-indirect-call-profiles
  • users/minglotus-6/spr/main.thinltotypeprofilingadd-annotated-vtable-guid-as-referenced-variables-in-per-function-summary-and-update-bitcode-writer-to-create-value-ids-for-these-referenced-vtables

They were not deleted when their associated pull requests were closed, only the non main. versions were.

Whatever the policy, we really need to disallow unreasonably long branch names either via a push hook or permission to delete them even if they are used in an active pr. If spr requires using unreasonably long names, then we shouldn’t use spr. It’s not ok to have git clone https://github.com/llvm/llvm-project.git not work on Windows.

6 Likes