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

CROSSTOOL docs: fix formatting errors #6974

Closed
wants to merge 4 commits into from

Conversation

laszlocsomor
Copy link
Contributor

No description provided.

Change-Id: I9b7e8561189124dbcb31aea4333bda4bf3f90681
@laszlocsomor laszlocsomor requested review from aehlig and hlopko and removed request for aehlig December 20, 2018 09:39
- Use "pre"-tag to format "Feature relationships" table, "Constraints"
  column
- Add an "Action" column for CROSSTOOL build variables' actions.

Change-Id: Ifbc296f713b5cc69a5b1836aaf9bffd8f9990a03
Change-Id: I7704bbb9b662ec3e25866abc71b7814074128ed3
@hlopko
Copy link
Member

hlopko commented Dec 20, 2018

Looks fantastic, thank you so much! Only nits for consistency. LGTM!

@laszlocsomor
Copy link
Contributor Author

Thank you for the swift review!

I see no comments though, did you intend to send none?

site/docs/crosstool-reference.md Outdated Show resolved Hide resolved
site/docs/crosstool-reference.md Outdated Show resolved Hide resolved
@hlopko
Copy link
Member

hlopko commented Dec 20, 2018

Hups :)

Change-Id: Idd8ad88f3ef8d09ac7901256aa66edc41fa62e68
@laszlocsomor
Copy link
Contributor Author

Done. I also removed the line-ending "," in the requires { ... } and with_feature { ... } messages.

@bazel-io bazel-io closed this in b2f0e26 Dec 20, 2018
@laszlocsomor laszlocsomor deleted the crosstool-docs branch December 20, 2018 11:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants