openedx / open-edx-proposals

Proposals for Open edX architecture, best practices and processes
http://open-edx-proposals.readthedocs.io/
Other
44 stars 32 forks source link

Use catalog-info.yaml for release metadata. Get rid of openedx.yaml #526

Closed feanil closed 10 months ago

feanil commented 11 months ago

OEP-55:ADR-4 in this PR has most of the context of what I'm proposing so I suggest reading that first. The rest is just a consequence of that decision.

nedbat commented 10 months ago

I'm realizing now (while watching the Quince cut) that we no longer have a way to say "maybe." What will our approach be? With openedx.yaml, the cookiecutter started the file off with maybe:true as a way to alert the release manager that no decision had been recorded.

Do we want to use the complete absence of the key to mean that?

feanil commented 10 months ago

@nedbat good point, I've created https://github.com/openedx/open-edx-proposals/pull/529 to address that.

kdmccormick commented 6 months ago

@feanil could you toss an issue on the BTR board so that we remember to apply this change before Redwood?

feanil commented 6 months ago

@kdmccormick I think nothing needs to change on the BTR side, once https://github.com/openedx/repo-tools/pull/467 lands, it should to work automatically.

kdmccormick commented 6 months ago

Ah, I wasn't aware of that PR. Cool, thanks.