Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Docs] Add DR002 for versioning in codegen #95

Merged

Conversation

feltech
Copy link
Member

@feltech feltech commented May 17, 2024

Part of #88. Consolidate the discussion, provoked by iterations of the design proposal in #90, into a decision record.

@feltech feltech self-assigned this May 17, 2024
@feltech feltech requested a review from a team as a code owner May 17, 2024 15:28
@feltech feltech force-pushed the work/88-drTraitVersioningNamespacing branch 2 times, most recently from 2e911a4 to 95bc073 Compare May 17, 2024 15:44
@elliotcmorris elliotcmorris requested review from themissingcow and removed request for foundrytom May 20, 2024 09:34
@feltech feltech requested a review from elliotcmorris May 24, 2024 15:42
@feltech feltech requested a review from elliotcmorris May 29, 2024 08:47
Part of OpenAssetIO#88. Consolidate the discussion, provoked by iterations of the
design proposal in OpenAssetIO#90, into a decision record.

Signed-off-by: David Feltell <david.feltell@foundry.com>
@feltech feltech force-pushed the work/88-drTraitVersioningNamespacing branch from 5f9355d to 71682e9 Compare May 31, 2024 10:29
@feltech feltech marked this pull request as draft May 31, 2024 10:52
@feltech
Copy link
Member Author

feltech commented May 31, 2024

Set to Draft to prevent accidental merge before community review (and squashing).

@feltech feltech marked this pull request as ready for review June 7, 2024 08:21
@feltech
Copy link
Member Author

feltech commented Jun 7, 2024

We gave it a week in Draft for community review. Let's assume no news is good news 😉

@feltech feltech merged commit 12bbfd7 into OpenAssetIO:main Jun 7, 2024
27 checks passed
@feltech feltech deleted the work/88-drTraitVersioningNamespacing branch June 7, 2024 12:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants