Skip to content

ci: add build & test#29

Merged
robert-zaremba merged 10 commits intomasterfrom
robert/ci-test
Jul 7, 2025
Merged

ci: add build & test#29
robert-zaremba merged 10 commits intomasterfrom
robert/ci-test

Conversation

@robert-zaremba
Copy link
Contributor

Description

Adding build and test CI jobs

Signed-off-by: Robert Zaremba <robert@zaremba.ch>
@robert-zaremba robert-zaremba self-assigned this Jul 7, 2025
Copilot AI review requested due to automatic review settings July 7, 2025 13:00
@robert-zaremba robert-zaremba requested a review from a team as a code owner July 7, 2025 13:00

This comment was marked as outdated.

@robert-zaremba robert-zaremba requested a review from Copilot July 7, 2025 13:49
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Adds a new GitHub Actions workflow for building and testing Move code in a dedicated container.

  • Introduces a .github/test.yml workflow that runs on merge_group and pull_request events
  • Uses tj-actions/changed-files to conditionally build and test only changed .move files
  • Configures concurrency to cancel in-progress runs on the same branch/workflow
Comments suppressed due to low confidence (3)

.github/test.yml:1

  • [nitpick] Consider renaming the workflow file to build-and-test.yml for clearer intent and consistency with its name.
name: Build and test

.github/test.yml:13

  • [nitpick] This job handles both build and test steps—consider renaming the job key (and display name) to build-and-test for clarity.
    test:

.github/test.yml:4

  • [nitpick] Add a brief comment explaining the merge_group trigger to help future maintainers understand its purpose.
    merge_group:

Signed-off-by: Robert Zaremba <robert@zaremba.ch>
Signed-off-by: Robert Zaremba <robert@zaremba.ch>
Signed-off-by: Robert Zaremba <robert@zaremba.ch>
Signed-off-by: Robert Zaremba <robert@zaremba.ch>
Signed-off-by: Robert Zaremba <robert@zaremba.ch>
Signed-off-by: Robert Zaremba <robert@zaremba.ch>
Signed-off-by: Robert Zaremba <robert@zaremba.ch>
Copy link
Contributor

@sczembor sczembor left a comment

Choose a reason for hiding this comment

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

lgtm.

Signed-off-by: Robert Zaremba <robert@zaremba.ch>
Signed-off-by: Robert Zaremba <robert@zaremba.ch>
@robert-zaremba robert-zaremba merged commit 2d2652a into master Jul 7, 2025
6 of 7 checks passed
@robert-zaremba robert-zaremba deleted the robert/ci-test branch July 7, 2025 17:24
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.

3 participants