Skip to content
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

The extension doesn't support the -experimental flag #29

Closed
thomaslevesque opened this issue Sep 26, 2016 · 6 comments
Closed

The extension doesn't support the -experimental flag #29

thomaslevesque opened this issue Sep 26, 2016 · 6 comments
Assignees
Milestone

Comments

@thomaslevesque
Copy link

thomaslevesque commented Sep 26, 2016

I use the -experimental flag to enable C# 6 features. Apparently the extension doesn't provide a way to pass this flag to cake.exe, so my Cake tasks fail to compile.

@agc93 agc93 self-assigned this Sep 26, 2016
@agc93 agc93 added this to the v0.2.0 milestone Sep 26, 2016
@agc93
Copy link
Member

agc93 commented Sep 26, 2016

I'll take a look at this, and I should be able to add it alongside the other options

agc93 added a commit to agc93/cake-vs that referenced this issue Sep 26, 2016
@thomaslevesque
Copy link
Author

btw, I was wondering: why do I still need "experimental" to use C# 6 features? C# 6 has been out for more than a year. I thought it was because the Roslyn scripting APIs weren't yet stable, but now they are...

@patriksvensson
Copy link
Member

We still target .NET 4.5 and that is using the older Roslyn bits. The experimental bits are not even the latest either but pre-release bits (but still a lot newer).

If you truly want the latest Roslyn bits, then you should use the Cake.CoreCLR package that contains cross platform Roslyn.

@thomaslevesque
Copy link
Author

Ah, thanks. I'll give it a try.

@gep13 gep13 closed this as completed in #32 Sep 29, 2016
gep13 added a commit that referenced this issue Sep 29, 2016
(GH-29) Add support for -experimental flag
@matkoch
Copy link

matkoch commented Dec 11, 2016

Is it possible to release this fix before all other issues for this milestone have been resolved? Things like auto-indention, drag&drop and auto debug can be easily workarounded. But with this feature missing, all developers are forced to write new, soon to be changed code. At least if they value the task runner explorer over invoking the script manually. #37 falls into the same category.

@agc93
Copy link
Member

agc93 commented Dec 11, 2016

@matkoch If you look at the PR for this issue, you can check out the AppVeyor build and you can download the VSIX directly from the build artifacts to locally install to get this fix right away.

We're working on getting a CI build available using MyGet (you can track that over at #18/#48 ), but for now each PR builds a VSIX you can use to get the latest changes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants