Skip to content

DOC-2096: Update diagrams.go to force scrolling on svg div blocks#73948

Merged
craig[bot] merged 1 commit intocockroachdb:masterfrom
nickvigilante:DOC-2096-crdb
Dec 17, 2021
Merged

DOC-2096: Update diagrams.go to force scrolling on svg div blocks#73948
craig[bot] merged 1 commit intocockroachdb:masterfrom
nickvigilante:DOC-2096-crdb

Conversation

@nickvigilante
Copy link
Copy Markdown
Contributor

Release note: None

@cockroach-teamcity
Copy link
Copy Markdown
Member

This change is Reviewable

@nickvigilante nickvigilante requested review from ericharmeling and ianjevans and removed request for jseldess December 17, 2021 00:51
Copy link
Copy Markdown

@ericharmeling ericharmeling left a comment

Choose a reason for hiding this comment

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

LGTM! Just tested the changes locally, and all SVGs generated exactly as they should.

IDK why the Bazel test is failing though. Maybe try rebasing on master and force-pushing to your fork branch? It should kick off the tests again.

@nickvigilante
Copy link
Copy Markdown
Contributor Author

@ericharmeling Can you bors this for me? Engineering doesn't let non-engineers use bors anymore.

@ericharmeling
Copy link
Copy Markdown

bors r+

@craig
Copy link
Copy Markdown
Contributor

craig bot commented Dec 17, 2021

Build succeeded:

@craig craig bot merged commit 52b398f into cockroachdb:master Dec 17, 2021
@nickvigilante nickvigilante deleted the DOC-2096-crdb branch January 25, 2022 19:21
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