-
Notifications
You must be signed in to change notification settings - Fork 216
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
fixed typos and closed open brackets (#807) #1126
Conversation
Update mfg_event.proto fixed typos and closed open brackets (google#807) fixed typos and closed open brackets (google#807)
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.
Thank you for these corrections. Everything looks good except the one comment I left.
We try to employ an 80 character line limit and that one line is now past 80 characters.
the _converter callable attribute only occurs once and the resulting protobuf | ||
is cached in memory on the instance. | ||
|
||
The upload callback will upload to mfg-inspector.com using the given | ||
username and authentication key (which should be the key data itself, not a | ||
filename or file). | ||
|
||
In typical productin setups, we *first* save the protobuf to disk then attempt | ||
In typical production setups, we *first* save the protobuf to disk then attempt |
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.
Would you mind tweaking this to get every line to fix into 80 characters?
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.
Would you review the change I made addressing to the comment?
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.
Ah sorry I should have been more clear. Lets move 1 word down one line at a time here. Basically what I mean is change this paragraph to be:
In typical production setups, we *first* save the protobuf to disk then
attempt to upload the protobuf to mfg-inspector. In the event of a network,
outage the result of the test run is available on disk and a separate process
can retry the upload when the network is available.
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.
Thank You I changed it as desired
addressed the comment "80 character/line"
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.
Thank you!
This change is