-
Notifications
You must be signed in to change notification settings - Fork 26
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
Merge latest master into dev to sync versions #286
Conversation
Release/0.11.0 to master
…-code-snippet Merge Getting Started code snippet into master
/** | ||
This file shows basic Vision SDK configuration steps. | ||
*/ | ||
// This file shows basic Vision SDK configuration steps. |
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'd leave the current formatting of example headers. Didn't notice that in PR to master.
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.
Formally, it was an empty doc comment which is incorrect.
I propose to merge the current PR anyway to sync the state, but we can use multiline comment
/*
This file shows basic Vision SDK configuration steps.
*/
here and in the rest of places later (while we're merging examples/code snippets).
Does that make sense?
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'm ok to merge it like this and adjust with other example PRs.
As for the comment look, we have already chosen the style for such comments (all the example files follow it), do you suggest to change it?
In scope of PR #283 a code snippet for "Getting Started guide" was introduced.
Then during merging the changes into a
master
branch, some minor fixes were made.Now we need to sync the minor fixes from
master
branch withdev
branch.Checks:
dev
branchLinked issues:
#283
#284