Skip to content

Refactor mt-sync-code.yml to match mt-build.yml structure with shared setup action#95

Draft
Copilot wants to merge 6 commits intomasterfrom
copilot/split-test-lint-jobs
Draft

Refactor mt-sync-code.yml to match mt-build.yml structure with shared setup action#95
Copilot wants to merge 6 commits intomasterfrom
copilot/split-test-lint-jobs

Conversation

Copy link
Contributor

Copilot AI commented Jan 29, 2026

  • Analyze the current structure of mt-sync-code.yml and mt-build.yml
  • Create separate MT-TEST-JOB and MT-LINT-JOB in mt-sync-code.yml
  • Update MT-SYNC-CODE-JOB to depend on both test and lint jobs
  • Move test_only.sh execution to MT-TEST-JOB
  • Move lint_only.sh execution to MT-LINT-JOB
  • Update artifact upload steps for each job appropriately
  • Fix comments in lint job to accurately reflect context
  • Rename job from MT-SYNC-CODE-DATA-JOB to MT-SYNC-CODE-JOB
  • Refactor to use ./.github/actions/setup with code-sync: true input
  • Add required environment variables (MT_BRANCH_NAME, MT_TARGET_BRANCH_NAME, MT_SHA)
  • Remove duplicate setup steps now handled by the shared setup action
  • Ensure the workflow continues to work correctly by validating the syntax

Summary

This PR refactors mt-sync-code.yml to match the structure of mt-build.yml by:

  • Splitting test and lint execution into separate parallel jobs
  • Using the shared ./.github/actions/setup action with code-sync: true to eliminate code duplication
  • Centralizing setup logic (checkout submodules, code sync, code setup, JDK, Gradle) in the reusable action

The workflow maintains all original functionality including code synchronization and the assemble step in the main sync job, while being more maintainable and consistent with other workflows in the repository.

Original prompt

This section details on the original issue you should resolve

<issue_title>CI > .github/workflows/mt-sync-code.yml should split test_only.sh and lint_only.sh in different jobs like .github/workflows/mt-build.yml</issue_title>
<issue_description></issue_description>

Comments on the Issue (you are @copilot in this section)


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI and others added 2 commits January 29, 2026 15:38
Co-authored-by: mmathieum <177998+mmathieum@users.noreply.github.com>
Co-authored-by: mmathieum <177998+mmathieum@users.noreply.github.com>
Copilot AI changed the title [WIP] Update CI to separate test and lint jobs in workflows Split test and lint into parallel jobs in mt-sync-code.yml Jan 29, 2026
Copilot AI requested a review from mmathieum January 29, 2026 15:44
@mmathieum mmathieum added the AI Made by/with AI (Gemini/Copilot) label Jan 29, 2026
Co-authored-by: mmathieum <177998+mmathieum@users.noreply.github.com>
Copy link
Member

Choose a reason for hiding this comment

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

@copilot please look at .github/workflows/mt-build.yml and re-use ./.github/actions/setup. It's basically the same workflow but with code-sync inputs as true and without the ASSEMBLE job.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Refactored to use ./.github/actions/setup with code-sync: true input in commit 01213c3. The workflow now matches the structure of mt-build.yml but with code-sync enabled and without the ASSEMBLE job (assemble is in the main sync job).

Co-authored-by: mmathieum <177998+mmathieum@users.noreply.github.com>
Copilot AI changed the title Split test and lint into parallel jobs in mt-sync-code.yml Refactor mt-sync-code.yml to match mt-build.yml structure with shared setup action Feb 5, 2026
Copilot AI requested a review from mmathieum February 5, 2026 14:23
@mmathieum mmathieum added the paused ⏸️ paused work label Feb 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

AI Made by/with AI (Gemini/Copilot) paused ⏸️ paused work

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CI > .github/workflows/mt-sync-code.yml should split test_only.sh and lint_only.sh in different jobs like .github/workflows/mt-build.yml

2 participants