-
Notifications
You must be signed in to change notification settings - Fork 563
[WIP] #r nuget improvements #586
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
d24b9fe to
cc66b52
Compare
NotebookExamples/Untitled.ipynb
Outdated
| @@ -0,0 +1,73 @@ | |||
| { | |||
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.
careful with untitled ntoebook
e11ac63 to
62d1271
Compare
| } | ||
|
|
||
| public ScriptOptions ScriptOptions { get; private set; } = | ||
| public ScriptOptions ScriptOptions { get; } = |
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.
Do we need this property as public?
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.
Probably not.
c5b82bc to
b61aadc
Compare
|
Should also fix https://github.com/dotnet/try/issues/458. |
0917a4c to
0bcdd01
Compare
|
Closing because #600 is better. |
This PR addresses #354, #583, #582, and #584.