Skip to content

Conversation

@chriseth
Copy link
Contributor

@chriseth chriseth commented Nov 8, 2021

No description provided.

@chriseth chriseth requested review from bshastry, cameel and hrkrshnn and removed request for hrkrshnn November 8, 2021 14:59
@chriseth chriseth force-pushed the irbreakingchangesvisible branch from 10b4c7f to ff1fb60 Compare November 8, 2021 15:01
Copy link
Contributor

@bshastry bshastry left a comment

Choose a reason for hiding this comment

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

LGTM. Is there a way to preview the web page to spot formatting errors?

@chriseth
Copy link
Contributor Author

chriseth commented Nov 8, 2021

Yes, go to the artifacts of the b_docs job: https://898698-40892817-gh.circle-artifacts.com/0/docs-html/ir-breaking-changes.html

bshastry
bshastry previously approved these changes Nov 8, 2021
@bshastry
Copy link
Contributor

bshastry commented Nov 8, 2021

Layout and text look good to me. Approving.

ekpyron
ekpyron previously approved these changes Nov 8, 2021
Copy link
Collaborator

@ekpyron ekpyron left a comment

Choose a reason for hiding this comment

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

I haven't checked for exhaustiveness of the docs, but the changes look good!

cameel
cameel previously approved these changes Nov 8, 2021
Copy link
Collaborator

@cameel cameel left a comment

Choose a reason for hiding this comment

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

I also haven't re-read the whole doc but the changes visible in the diff look reasonable.

@chriseth chriseth dismissed stale reviews from cameel, ekpyron, and bshastry via df85d26 November 8, 2021 16:23
@chriseth chriseth force-pushed the irbreakingchangesvisible branch from 227bb04 to df85d26 Compare November 8, 2021 16:23
@chriseth chriseth merged commit f095442 into develop Nov 8, 2021
@chriseth chriseth deleted the irbreakingchangesvisible branch November 8, 2021 16:28
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.

5 participants