Skip to content

Update to use the new build container #26

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.

Sign up for GitHub

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

Jump to bottom
Merged
EkimChau merged 4 commits into from
Jul 31, 2025
Merged

Update to use the new build container #26

EkimChau merged 4 commits into from
Jul 31, 2025

Conversation

@tiny-ben-tran
Copy link
Contributor

@tiny-ben-tran tiny-ben-tran commented Jul 21, 2025 •
edited
Loading

Description of changes:

  • Updated Jenkinsfile to use the new build function
  • Upgraded bedrock-client and bedrock-server to v15 which has remote features that used by the new build function

@tiny-ben-tran tiny-ben-tran requested review from a team, Skylite73, TheSpyder, jscasca and ltrouton and removed request for a team July 22, 2025 07:01
@tiny-ben-tran tiny-ben-tran marked this pull request as ready for review July 22, 2025 23:19
@tiny-ben-tran tiny-ben-tran requested a review from a team as a code owner July 22, 2025 23:19
@tiny-ben-tran tiny-ben-tran requested review from ArvinJ-H, kimwoodfield, shanmen-tiny and spocke and removed request for a team July 22, 2025 23:19
kimwoodfield
kimwoodfield approved these changes Jul 24, 2025
View reviewed changes
Copy link

@kimwoodfield kimwoodfield left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for moving this across!

@EkimChau EkimChau merged commit 14ea2b9 into main Jul 31, 2025
6 checks passed
@EkimChau EkimChau deleted the feature/update_ci_build branch July 31, 2025 03:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jscasca jscasca jscasca approved these changes

@kimwoodfield kimwoodfield kimwoodfield approved these changes

@TheSpyder TheSpyder Awaiting requested review from TheSpyder TheSpyder was automatically assigned from tinymce/editor-platform-team

@ltrouton ltrouton Awaiting requested review from ltrouton ltrouton was automatically assigned from tinymce/editor-platform-team

@Skylite73 Skylite73 Awaiting requested review from Skylite73 Skylite73 was automatically assigned from tinymce/editor-platform-team

@spocke spocke Awaiting requested review from spocke spocke is a code owner automatically assigned from tinymce/tinymce-reviewers

@ArvinJ-H ArvinJ-H Awaiting requested review from ArvinJ-H

@shanmen-tiny shanmen-tiny Awaiting requested review from shanmen-tiny

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@tiny-ben-tran @jscasca @kimwoodfield @EkimChau