futures-rs icon indicating copy to clipboard operation
futures-rs copied to clipboard

Warn when PR is opened for 0.3/0.1 branch

Open taiki-e opened this issue 1 year ago • 1 comments

This would help avoid problems like https://github.com/rust-lang/futures-rs/pull/2801#issuecomment-1819252187.

rust-lang/rust already has a mechanism for this (https://github.com/rust-lang/rust/pull/118607#issuecomment-1838839291), but I don't know how to enable it for this repo.

taiki-e avatar Dec 07 '23 17:12 taiki-e

I think having explicit mention of it in CONTRIBUTING.md (or even any documentation about the branching/backporting policy at all) would be a good start. I explicitly looked for some, didn't find any, found that HEAD pointed at 0.4, and figured I should target my bugfix to the release branch.

In other words, it wasn't exactly an accident: it was my genuine best guess at how to get a bugfix merged into a previous release line in the absence of documentation, given that I was uncertain that a fix to master would get cherry-picked to a release branch.

edef1c avatar Feb 05 '24 21:02 edef1c

cc @Kobzol: Are there any options for doing this in rust-lang/team or somewhere? I was going to refer to what rust-lang/rust does in rust-lang/team, but rust.toml doesn't seem to exist in the dierctry that futures-rs.toml exist, so I'm not sure what to do.

taiki-e avatar Mar 09 '24 08:03 taiki-e

Not in rust-lang/team, this is a functionality of rustbot/triagebot. Also, rust-lang/rust isn't managed by team, it will probably be the last repo to be added due to its complexity :laughing:

You need to configure this in the triagebot file.

Kobzol avatar Mar 09 '24 08:03 Kobzol

You need to configure this in the triagebot file.

Thanks! I opened a PR to enable it.

taiki-e avatar Mar 09 '24 08:03 taiki-e