Skip to content

Move to a more recent version of argu - #3112

Merged
forki merged 5 commits into
fsprojects:masterfrom
vbfox:new_argu
Mar 13, 2018
Merged

forki merged 5 commits into
fsprojects:masterfrom
vbfox:new_argu

Conversation

@vbfox

@vbfox vbfox commented Mar 12, 2018

Copy link
Copy Markdown
Contributor

To get a few perf optimizations, waiting for the next ones

Comment thread paket.dependencies
clitool dotnet-mergenupkg

github fsharp/FAKE src/app/FakeLib/Globbing/Globbing.fs
github fsharp/FAKE:0341a2e614eb2a7f34607cec914eb0ed83ce9add src/app/FakeLib/Globbing/Globbing.fs

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

The file moved to /src/legacy/FakeLib/Globbing/Globbing.fs that was probably the problem?

Comment thread src/Paket/Commands.fs
| From_Bootstrapper -> "call coming from the '--run' feature of the bootstrapper"

let commandParser = ArgumentParser.Create<Command>(programName = "paket", errorHandler = new ProcessExiter())
let commandParser = ArgumentParser.Create<Command>(programName = "paket", errorHandler = new ProcessExiter(), checkStructure = false)

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Just out of curiosity what is the impact of this? Is there an impact?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

On performance it's faster :)

It also doesn't check the argument DU for incoherences, but it's forced in unit tests so it would fail there.

@forki
forki merged commit fa80569 into fsprojects:master Mar 13, 2018
@forki

forki commented Mar 13, 2018

Copy link
Copy Markdown
Member

Thanks

@forki

forki commented Mar 13, 2018

Copy link
Copy Markdown
Member

released in 5.150.0

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