Skip to content

fix(ci): format build and test command for better readability#763

Merged
Leomrlin merged 1 commit intoapache:masterfrom
SYaoJun:0304_ci
Mar 20, 2026
Merged

fix(ci): format build and test command for better readability#763
Leomrlin merged 1 commit intoapache:masterfrom
SYaoJun:0304_ci

Conversation

@SYaoJun
Copy link
Copy Markdown
Contributor

@SYaoJun SYaoJun commented Mar 4, 2026

What changes were proposed in this pull request?

fix: #762
The current command does not support multi-line input.

How was this PR tested?

  • Tests have Added for the changes
  • Production environment verified

@SYaoJun
Copy link
Copy Markdown
Contributor Author

SYaoJun commented Mar 6, 2026

The problem introduced in https://github.com/apache/geaflow/pull/637/changes, which split one line command into two lines.

@Leomrlin
Copy link
Copy Markdown
Contributor

Leomrlin commented Mar 9, 2026

Since this PR only changed the YML file and CI settings don’t run related tests, can we add a code change and try running the CI tests?

@SYaoJun
Copy link
Copy Markdown
Contributor Author

SYaoJun commented Mar 9, 2026

Since this PR only changed the YML file and CI settings don’t run related tests, can we add a code change and try running the CI tests?

ok

@SYaoJun
Copy link
Copy Markdown
Contributor Author

SYaoJun commented Mar 9, 2026

I've updated the code; could you help trigger the CI? @Leomrlin

Copy link
Copy Markdown
Contributor

@Leomrlin Leomrlin left a comment

Choose a reason for hiding this comment

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

LGTM

@Leomrlin Leomrlin merged commit f8be722 into apache:master Mar 20, 2026
2 checks passed
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.

[CI] ci.yml has a syntax error

2 participants