Re: [PATCH] Documentation: Architecture Decision Record

Alyssa Ross
2022-07-22T11:37:23Z

[-- Attachment #1: Type: text/plain, Size: 887 bytes --]

On Wed, Jul 13, 2022 at 02:04:31PM +0300, Ville Ilvonen wrote:
> On Wed, Jul 13, 2022 at 12:53 PM Alyssa Ross <hi@alyssa.is> wrote:
> >
> > I've thought about this a bit more, and decided that it's fine to have
> > proposed ADRs in the repo. I was concerned about the in-between state
> > an ADR could find itself in when it's been submitted but I haven't
> > applied the patch yet, but having thought about it more, I don't think
> > having a stage of "being accepted for discussion" is a bad thing.
>
> Ok. This patch addresses all your review comments.

Thanks. I've applied the patch.

I still have some reservations about the ADRs — I'm still a bit worried
about the proposed ADRs being in the repo making people hesitant to
suggest changes to them, and I'm not sure yet how they fit into the rest
of the documentation — but let's give them a go, as a trial.