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

[WIP] csx build script spike #232

Closed
wants to merge 14 commits into from
Closed

Conversation

adamralph
Copy link
Member

@adamralph adamralph commented Sep 16, 2016

connects to #220

@adamralph adamralph force-pushed the spike-build-script-csx branch from 6c2432e to eea75c2 Compare September 16, 2016 11:48
@adamralph
Copy link
Member Author

Unfortunately it seems like VS Intellisense doesn't respect #r in a csx file. Raised dotnet/roslyn#13856

@adamralph adamralph force-pushed the spike-build-script-csx branch from dc90c25 to 9610ecc Compare September 18, 2016 01:29
@adamralph adamralph force-pushed the spike-build-script-csx branch from 9610ecc to 8d147f6 Compare September 18, 2016 01:35
@adamralph
Copy link
Member Author

I wanted to split out the boiler plate for running and writing tasks into two separate csx files, and bring them into build.csx using #load but sadly VS Intellisense doesn't support #load. Raised dotnet/roslyn#13888

Also raised dotnet/roslyn#13886 and dotnet/roslyn#13887 for other csx Intellisense woes. Generally it seems to be rather unfinished and buggy.

@adamralph
Copy link
Member Author

Closing spike.

@adamralph adamralph closed this Sep 29, 2016
@adamralph adamralph deleted the spike-build-script-csx branch September 30, 2016 17:51
@filipw
Copy link

filipw commented May 3, 2017

use VS Code 😄

@adamralph
Copy link
Member Author

👍 I've been having much better results with VS Code recently. The C# script support really seems to be going places.

@filipw
Copy link

filipw commented May 4, 2017

great. the upcoming release will have "go to metadata" support from CSX.
Also all other general C# work (like i.e .OmniSharp/omnisharp-roslyn#840) is now automatically available in CSX too.

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