ejoffe / spr

Stacked Pull Requests on GitHub
MIT License
714 stars 65 forks source link

Warning message on PR descriptions needs an update #404

Open FabriceCastel opened 2 months ago

FabriceCastel commented 2 months ago

It seems that it is in fact FINE to:

I assume the way things can go "bad" is if you accidentally merge a branch that's targeting one of the automatically created spr branches instead of main, for example.

The error message that gets tacked on at the end of PR messages should be updated to reflect this. Either of the following seems reasonable to me: