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

diagrams: Diagrams job no longer fails when no diagrams change #89753

Merged
merged 1 commit into from
Oct 28, 2022
Merged

diagrams: Diagrams job no longer fails when no diagrams change #89753

merged 1 commit into from
Oct 28, 2022

Conversation

nickvigilante
Copy link
Contributor

If there are PRs that don't affect the diagrams or their related files, the Publish SQL Grammar Diagrams job will fail, because git commit produces a non-zero exit code if there are no files changed in the working directory. This fixes that problem.

Release note: None

Fixes DOC-5642

@cockroach-teamcity
Copy link
Member

This change is Reviewable

@rail rail self-requested a review October 26, 2022 14:13
build/teamcity-diagram-generation.sh Outdated Show resolved Hide resolved
If there are PRs that don't affect the diagrams or their related files,
the Publish SQL Grammar Diagrams job will fail, because `git commit`
produces a non-zero exit code if there are no files changed in the
working directory. This fixes that problem.

Release note: None

Fixes DOC-5642
@nickvigilante
Copy link
Contributor Author

bors r+

@craig
Copy link
Contributor

craig bot commented Oct 28, 2022

Build succeeded:

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