-
Notifications
You must be signed in to change notification settings - Fork 4.6k
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
Allows to pass the settings.json file location via the --settings argument #2668
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.
Wasn't aware of this, thanks!
just updated the pull request according to your suggestions :) |
Thanks :) One last thing which will be good to do would be to squash the commits into 1, that keeps the history cleaner and makes life easy for the maintainers |
…ument as a path in addition to passing its content. Also updated documentation, which did not list the other loading possibilities added with microsoft#715.
@rajat2004 done ;) |
/azp run microsoft.AirSim |
Azure Pipelines successfully started running 1 pipeline(s). |
Allows to pass the settings.json file location via the --settings argument as a path in addition to passing its content.
Also updated documentation, which did not list the other loading possibilities added with #715.