-
Notifications
You must be signed in to change notification settings - Fork 249
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
fix typos #1192
base: master
Are you sure you want to change the base?
fix typos #1192
Conversation
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.
The change in assembly.rs
incorrectly alters the sense of the comment there. Please revert that hunk.
@@ -752,7 +752,7 @@ impl<'c> Translation<'c> { | |||
// Extract expression | |||
let in_expr = in_expr.map(|(i, operand)| (i, operand.expression)); | |||
|
|||
// For inouts, change the dirspec to include 'in' |
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.
NAK. An inout is not an input, it's a bidirectional (in and out) argument; see above where we also mention that we are handling inout arguments.
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.
Oh whoops I didn't see that.
I have also added a .typos.toml , config for https://crates.io/crates/typos-cli ; happy to remove if not desired, or move into a .config/ in order to reduce top level files. |
No description provided.