-
Notifications
You must be signed in to change notification settings - Fork 394
Discover settings file from given path #614
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
Conversation
README.md, line 221 at r1 (raw file):
Point to the Settings file in the release branch - aka Master Comments from Reviewable |
Engine/Commands/InvokeScriptAnalyzerCommand.cs, line 38 at r1 (raw file):
Minor: This help uri redirects to GitHub page. Wondering if a fw link is really required OR we can point to Github directly. Comments from Reviewable |
Reviewed 8 of 8 files at r1. Comments from Reviewable |
Review status: 7 of 8 files reviewed at latest revision, 2 unresolved discussions. README.md, line 221 at r1 (raw file):
|
Reviewed 1 of 1 files at r2. Comments from Reviewable |
Comments from Reviewable |
@raghushantha Thanks! |
Partially resolves #518
This change is