-
Notifications
You must be signed in to change notification settings - Fork 158
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
Docs language edit - Part 1 #830
Conversation
One question regarding the capitalization of the "compiler" in |
when it's code, yes most of the capitalization is not chosen by the doc writter, but by some rules or filenames in the langage. Could you tell me precise in which file and place you see that, please? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I marked the inconsistant capitalizations of "compiler" and "circuit.
Could you verify if they are correct?
Sorry @yuxizama I've a look early next week |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thx @yuxizama . Some proposed changes.
@bcm-at-zama |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Minor typo please
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thx!
@BourgerieQuentin : we don't authorize squash and merge in Concrete? here, it would be useful, we're not going to ask Yuxi to squash locally on her terminal |
@yuxizama will squash! |
2032465
to
07937f8
Compare
I've rephrased some sentences to increase the readability. Please check to ensure that my modification does not alter the technical implications.
Feedback is welcomed!