Skip to content

Conversation

@JoeRobich
Copy link
Member

@JoeRobich JoeRobich commented Aug 15, 2019

FolderWorkspace treats code files beneath the folder as part of the Solution. This workspace wouldn't be suitable to future analyzer/codefix work since semantic information wouldn't be available, but it does suffice to provide a faster path for whitespace formatting.

Opt-in to this experience by using the --folder option instead of --workspace.

  • Add FolderWorkspace
  • Add commandline option to use FolderWorkspace
  • Update readme.md

@JoeRobich
Copy link
Member Author

@jmarolf please take a look

@JoeRobich JoeRobich force-pushed the add-folderworkspace branch 2 times, most recently from bf91875 to 9253b41 Compare August 27, 2019 18:15
@jmarolf
Copy link
Contributor

jmarolf commented Aug 27, 2019

I'll review this today

@JoeRobich JoeRobich force-pushed the add-folderworkspace branch from 9253b41 to cf6238c Compare August 27, 2019 19:59
@jmarolf
Copy link
Contributor

jmarolf commented Aug 27, 2019

@JoeRobich what is the perf difference here?

@JoeRobich JoeRobich force-pushed the add-folderworkspace branch from cf6238c to fae06aa Compare August 27, 2019 20:17
@JoeRobich
Copy link
Member Author

@jmarolf for dotnet/format it goes from being ~4s down to ~1.5s

@JoeRobich JoeRobich force-pushed the add-folderworkspace branch 16 times, most recently from f4a57d4 to 1c45e81 Compare August 27, 2019 22:58
@JoeRobich JoeRobich force-pushed the add-folderworkspace branch 3 times, most recently from cd02308 to 63b8fb0 Compare August 27, 2019 23:20
@JoeRobich JoeRobich force-pushed the add-folderworkspace branch from 63b8fb0 to 04670cf Compare August 27, 2019 23:43
@JoeRobich JoeRobich merged commit aed0c64 into dotnet:master Aug 28, 2019
@JoeRobich JoeRobich deleted the add-folderworkspace branch March 5, 2021 21:01
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.

2 participants