sourcegraph / sourcegraph-public-snapshot

Code AI platform with Code Search & Cody
https://sourcegraph.com
Other
10.12k stars 1.29k forks source link

batches: support renaming a batch change #48085

Open courier-new opened 1 year ago

courier-new commented 1 year ago

It's common for a batch change to evolve over time. With that, its title should be as flexible. Users don't realize they're committing to a more-or-less permanent title when they first start their batch change, and it's painful to have to create a new one and copy all your work over just to rename it, which is the only way to achieve this today.

Feedback from Olaf Geirsson when running server-side for the first time:

Small feature request: it would be nice if I could rename a batch change. I first used the name "Openrewrite" but on a second thought I wish I had used the name "openrewrite-maven" or something like that

There's a couple complications to doing this.

The first is that we currently match batch specs to batch changes based on their name. We'd need to make sure when updating the batch change name in the database, we also update the name in all batch specs that are associated with it.

The second is that we'd be breaking the link back to the parent batch change that appears in pull request descriptions:

Image

There's a couple ways we could get around this:

github-actions[bot] commented 1 year ago

Hey, @sourcegraph/batchers (@eseliger @courier-new @adeola-ak @BolajiOlajide @Piszmog @malomarrec @chrispine @danielmarquespt) - we have been mentioned. Let's take a look.

courier-new commented 1 year ago

As with all of the server-side UX enhancement issues, I think these are best saved for another day soon when we can prioritize a whole slew of UX improvements. I'd vote backlog on this for now.