Skip to content
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.

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

fix: fix github action to check untracked files #19

Merged
merged 3 commits into from
Mar 24, 2023

Conversation

vinland-avalon
Copy link
Contributor

I hereby agree to the terms of the GreptimeDB CLA

What's changed and what's your intention?

Fix github action to check untracked files.

The git diff --exit-code cannot found difference of those untracked files. So I used git status instead.

@vinland-avalon vinland-avalon force-pushed the fix/github-action-check-untracked-files branch 7 times, most recently from 440bc9f to 83f8aaa Compare March 23, 2023 09:30
@vinland-avalon vinland-avalon force-pushed the fix/github-action-check-untracked-files branch from 83f8aaa to f86480c Compare March 23, 2023 09:32
@vinland-avalon vinland-avalon force-pushed the fix/github-action-check-untracked-files branch from a88d499 to e0aa307 Compare March 23, 2023 09:43
@yuanbohan yuanbohan requested a review from zyy17 March 23, 2023 09:45
@vinland-avalon vinland-avalon force-pushed the fix/github-action-check-untracked-files branch from e0aa307 to ef4f65a Compare March 23, 2023 09:47
@vinland-avalon vinland-avalon force-pushed the fix/github-action-check-untracked-files branch from ef4f65a to 0f46f63 Compare March 23, 2023 09:52
.github/workflows/main.yml Outdated Show resolved Hide resolved
@vinland-avalon vinland-avalon force-pushed the fix/github-action-check-untracked-files branch 2 times, most recently from eba9545 to 111c1c9 Compare March 23, 2023 12:05
@vinland-avalon vinland-avalon force-pushed the fix/github-action-check-untracked-files branch from 111c1c9 to 0402fcd Compare March 23, 2023 12:08
@vinland-avalon
Copy link
Contributor Author

vinland-avalon commented Mar 23, 2023

@zyy17 After refactoring, I've tried it twice, one with wrong generated files, the other with right ones. And the github actions successfully checked them: the former failed and the latter succeed. PTAL

Copy link
Contributor

@zyy17 zyy17 left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@yuanbohan yuanbohan left a comment

Choose a reason for hiding this comment

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

LGTM

@yuanbohan yuanbohan merged commit 9eaf95e into main Mar 24, 2023
@yuanbohan yuanbohan deleted the fix/github-action-check-untracked-files branch March 24, 2023 01:37
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