这是indexloc提供的服务,不要输入任何密码
Skip to content

Conversation

@jonsequitur
Copy link
Contributor

@jonsequitur jonsequitur commented Nov 1, 2019

This PR addresses #354, #583, #582, and #584.

@jonsequitur jonsequitur changed the title [WIP] #r nuget perf improvements [WIP] #r nuget improvements Nov 1, 2019
@@ -0,0 +1,73 @@
{
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

careful with untitled ntoebook

@jonsequitur jonsequitur force-pushed the octothorpe-r-perf branch 2 times, most recently from e11ac63 to 62d1271 Compare November 2, 2019 17:43
}

public ScriptOptions ScriptOptions { get; private set; } =
public ScriptOptions ScriptOptions { get; } =
Copy link
Member

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?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Probably not.

@eerhardt
Copy link
Member

eerhardt commented Nov 4, 2019

Should also fix https://github.com/dotnet/try/issues/458.

@jonsequitur jonsequitur closed this Nov 5, 2019
@jonsequitur jonsequitur reopened this Nov 5, 2019
@jonsequitur
Copy link
Contributor Author

Closing because #600 is better.

@jonsequitur jonsequitur closed this Nov 6, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants