Request to fix docs mistake in Boost.Outcome

I know the beta release is happening, but when it's out and master branch is reopened can I get permission to merge a mistake in documentation fixed by https://github.com/ned14/outcome/issues/280? Niall

On Thu, Jul 13, 2023 at 6:04 AM Niall Douglas via Boost
I know the beta release is happening, but when it's out and master branch is reopened can I get permission to merge a mistake in documentation fixed by https://github.com/ned14/outcome/issues/280?
I thought documentation changes were unpermissioned and all-you-can-eat? Thanks

On 13/07/2023 14:16, Vinnie Falco wrote:
On Thu, Jul 13, 2023 at 6:04 AM Niall Douglas via Boost
wrote: I know the beta release is happening, but when it's out and master branch is reopened can I get permission to merge a mistake in documentation fixed by https://github.com/ned14/outcome/issues/280?
I thought documentation changes were unpermissioned and all-you-can-eat?
Marshall has gotten annoyed with me in the past for changing the Outcome root SHA without telling him first, so now I tell him first. Niall

On Jul 13, 2023, at 6:20 AM, Niall Douglas via Boost
On 13/07/2023 14:16, Vinnie Falco wrote:
On Thu, Jul 13, 2023 at 6:04 AM Niall Douglas via Boost
wrote: I know the beta release is happening, but when it's out and master branch is reopened can I get permission to merge a mistake in documentation fixed by https://github.com/ned14/outcome/issues/280?
I thought documentation changes were unpermissioned and all-you-can-eat?
Marshall has gotten annoyed with me in the past for changing the Outcome root SHA without telling him first, so now I tell him first.
Marshall would like to believe :-) that he hasn’t been annoyed about doc fixes in the past. — Marshall P.S. I recall objecting to a set of changes that included doc fixes as well as other stuff.

On 13/07/2023 14:45, Marshall Clow wrote:
On Jul 13, 2023, at 6:20 AM, Niall Douglas via Boost
wrote: On 13/07/2023 14:16, Vinnie Falco wrote:
On Thu, Jul 13, 2023 at 6:04 AM Niall Douglas via Boost
wrote: I know the beta release is happening, but when it's out and master branch is reopened can I get permission to merge a mistake in documentation fixed by https://github.com/ned14/outcome/issues/280?
I thought documentation changes were unpermissioned and all-you-can-eat?
Marshall has gotten annoyed with me in the past for changing the Outcome root SHA without telling him first, so now I tell him first.
Marshall would like to believe :-) that he hasn’t been annoyed about doc fixes in the past.
— Marshall
P.S. I recall objecting to a set of changes that included doc fixes as well as other stuff.
Correct. Has master branch reopened for merge now the beta is out? Niall

On Jul 14, 2023, at 2:27 AM, Niall Douglas via Boost
On 13/07/2023 14:45, Marshall Clow wrote:
On Jul 13, 2023, at 6:20 AM, Niall Douglas via Boost
wrote: On 13/07/2023 14:16, Vinnie Falco wrote:
On Thu, Jul 13, 2023 at 6:04 AM Niall Douglas via Boost
wrote: I know the beta release is happening, but when it's out and master branch is reopened can I get permission to merge a mistake in documentation fixed by https://github.com/ned14/outcome/issues/280?
I thought documentation changes were unpermissioned and all-you-can-eat?
Marshall has gotten annoyed with me in the past for changing the Outcome root SHA without telling him first, so now I tell him first.
Marshall would like to believe :-) that he hasn’t been annoyed about doc fixes in the past.
— Marshall
P.S. I recall objecting to a set of changes that included doc fixes as well as other stuff.
Correct.
Has master branch reopened for merge now the beta is out?
Yes it has. — Marshall

On 14/07/2023 16:05, Marshall Clow via Boost wrote:
Has master branch reopened for merge now the beta is out?
Yes it has.
Excellent. Can I get permission to merge a few small bug fixes plus docs fixes? https://github.com/boostorg/outcome/commits/develop 1. Changes just before the beta window breaks build on C++ 14 in certain corner cases, fixed. 2. Experimental Outcome Boost result did not compile due to changes to status code by LEWG, fixed. 3. Docs fixes also due to the LEWG renaming breaking docs examples. Niall

On Jul 15, 2023, at 2:56 PM, Niall Douglas via Boost
On 14/07/2023 16:05, Marshall Clow via Boost wrote:
Has master branch reopened for merge now the beta is out?
Yes it has.
Excellent.
Can I get permission to merge a few small bug fixes plus docs fixes?
https://github.com/boostorg/outcome/commits/develop
1. Changes just before the beta window breaks build on C++ 14 in certain corner cases, fixed.
2. Experimental Outcome Boost result did not compile due to changes to status code by LEWG, fixed.
3. Docs fixes also due to the LEWG renaming breaking docs examples.
Go ahead. — Marshall

On 7/13/23 16:20, Niall Douglas via Boost wrote:
On 13/07/2023 14:16, Vinnie Falco wrote:
On Thu, Jul 13, 2023 at 6:04 AM Niall Douglas via Boost
wrote: I know the beta release is happening, but when it's out and master branch is reopened can I get permission to merge a mistake in documentation fixed by https://github.com/ned14/outcome/issues/280?
I thought documentation changes were unpermissioned and all-you-can-eat?
Marshall has gotten annoyed with me in the past for changing the Outcome root SHA without telling him first, so now I tell him first.
I believe, there is a time frame during archive preparation when *no* changes to master are permitted. Other than that, changes to docs don't need a permission.
participants (4)
-
Andrey Semashev
-
Marshall Clow
-
Niall Douglas
-
Vinnie Falco