Add ADR for Shared CNCF Workflow Specification Editor#1144
Add ADR for Shared CNCF Workflow Specification Editor#1144ricardozanini merged 1 commit intoserverlessworkflow:mainfrom
Conversation
lornakelly
force-pushed
the
adr-shared-editor
branch
from
7aa3eae to
ed5988e
Compare
There was a problem hiding this comment.
Pull request overview
Adds an Architecture Decision Record (ADR) documenting the proposal for a shared, spec-driven CNCF Workflow Specification editor and a multi-maintainer collaboration/governance model (per issue #1131).
Changes:
- Introduces an
adr/directory and adds an initial ADR for the shared editor initiative. - Documents governance/ownership, MVP scope, tooling choices, and anticipated milestones for delivery.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
cdavernas
left a comment
cdavernas
left a comment
There was a problem hiding this comment.
Aside from Copilot's nitpicks, LGTM! Thanks ❤️
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 1 out of 1 changed files in this pull request and generated 4 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
lornakelly
force-pushed
the
adr-shared-editor
branch
from
14e656a to
31729b2
Compare
lornakelly
force-pushed
the
adr-shared-editor
branch
from
eb3e310 to
d8a5f25
Compare
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
lornakelly
force-pushed
the
adr-shared-editor
branch
from
d8a5f25 to
b584180
Compare
|
Thanks @cdavernas updated PR with Copilot changes |
|
@lornakelly you must sign your commits. Can you squash them and sign it? |
lornakelly
force-pushed
the
adr-shared-editor
branch
6 times, most recently
from
5cf2a98 to
27a4cdb
Compare
|
@lornakelly I think you added some commits from other issues by mistake. Can you please review your git log? Probably messed them up while squashing. |
…aboration Model ADR Signed-off-by: lornakelly <lornakelly88@gmail.com> Signed-off-by: Dmitriy <dimastbk@proton.me> Co-authored-by: Charles d'Avernas <charles.davernas@neuroglia.io>
lornakelly
force-pushed
the
adr-shared-editor
branch
from
9649281 to
82db57b
Compare
Please specify parts of this PR update:
Discussion or Issue link:
Closes #1131
What this PR does:
Create folder for ADRs and add the proposed ADR for a Shared CNCF Workflow Specification Editor & Multi-Maintainer Collaboration Model