Skip to content

infra: validate git status after commands#11815

Merged
romani merged 1 commit into
checkstyle:masterfrom
romani:infra-git-diff
Jul 3, 2022
Merged

infra: validate git status after commands#11815
romani merged 1 commit into
checkstyle:masterfrom
romani:infra-git-diff

Conversation

@romani

@romani romani commented Jul 2, 2022

Copy link
Copy Markdown
Member

from #11806

update to catch unstaged files

@romani

romani commented Jul 3, 2022

Copy link
Copy Markdown
Member Author

it works:
https://cloud.drone.io/checkstyle/checkstyle/11062/3/4

[INFO] BUILD SUCCESS

[INFO] ------------------------------------------------------------------------

[INFO] Total time:  44.002 s

[INFO] Finished at: 2022-07-03T00:01:12Z

[INFO] ------------------------------------------------------------------------

version:10.4-SNAPSHOT

+ ./.ci/validation.sh git-diff

Please clean up or update .gitattributes file.

Git status output:

Top 300 lines of diff:

On branch master

Your branch is ahead of 'origin/master' by 2 commits.

  (use "git push" to publish your local commits)


Untracked files:

  (use "git add <file>..." to include in what will be committed)

	dependency-reduced-pom.xml


nothing added to commit but untracked files present (use "git add" to track)

@romani romani merged commit fcd1e80 into checkstyle:master Jul 3, 2022
@romani romani deleted the infra-git-diff branch July 3, 2022 00:35
Comment thread .ci/validation.sh
cd .ci-temp/pgjdbc
# pgjdbc easily damage build, we should use stable versions
git checkout "261181f31c0eb""e2deb593c1cc51174898ad6c50c"
git checkout "417c9a2354ad""c3d2c80f84b0a5059ce""ad92e7c2b"

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Why?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Ups, I am not sure how it happened, I will revert such changes

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Pushed

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.

2 participants