Skip to content

Conversation

@kdmccormick
Copy link
Member

@kdmccormick kdmccormick commented Oct 30, 2025

We only branch and tag top-level backend services, frontend apps, and non-default Open edX plugins.
This, on the other hand, is a package that gets installed into edx-platform. We shouldn't tag it, because its release/ulmo branch is not guaranteed to line up with the openedx-forum version installed by the release/ulmo branch of edx-platform.

After merging, we should delete the release/ulmo branch.

@ormsbee , @farhaanbukhsh

We only branch and tag top-level backend services,
frontend apps, and non-default Open edX plugins.
This, on the other hand, is a package that gets installed into
edx-platform. We shouldn't tag it, because its `release/ulmo` branch is not guaranteed to line up with the openedx-forum version installed by the
`release/ulmo` branch of edx-platform.

After merging, we should delete the `release/ulmo` tag.
@kdmccormick kdmccormick requested a review from ormsbee October 30, 2025 17:49
@ormsbee ormsbee merged commit fb17be3 into master Oct 30, 2025
24 of 26 checks passed
@ormsbee ormsbee deleted the kdmccormick/no-tag branch October 30, 2025 19:04
@farhaanbukhsh
Copy link
Member

farhaanbukhsh commented Oct 31, 2025

thank you for handling this @kdmccormick @ormsbee can you remove the branches as well or should I do that?

@kdmccormick
Copy link
Member Author

already did it :) we also rm'd the branch on tinymce-language-selector.

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.

4 participants