Skip to content

Conversation

@glennblock
Copy link
Contributor

Fix for #1133

@@ -1,9 +1,18 @@
//------------------------------------------------------------------------------
Copy link
Member

Choose a reason for hiding this comment

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

this file should be reverted, the original even says:

Do not manually edit this file

@filipw
Copy link
Member

filipw commented Feb 17, 2016

not sure what to do with scriptcs.visualbasic?

@glennblock
Copy link
Contributor Author

Hmm I think my changes got overwritten when I pulled Dev. I had renamed the
module already....
On Wed, Feb 17, 2016 at 7:03 AM Filip W notifications@github.com wrote:

not sure what to do with scriptcs.visualbasic?


Reply to this email directly or view it on GitHub
#1134 (comment).

@glennblock
Copy link
Contributor Author

@filipw we should remove VB probably. Maybe you should release it as a sep module?

@filipw
Copy link
Member

filipw commented Feb 27, 2016

yeah let's remove it, can you please update this branch to revert the module rename

@glennblock
Copy link
Contributor Author

@filipw thanks for the feedback and for catching the issues. All issues have been fixed and I removed the VB module.

filipw added a commit that referenced this pull request Feb 28, 2016
Renames ScriptCs.CSharp to ScriptCs.Engine.Roslyn
@filipw filipw merged commit c2d53cb into scriptcs:dev Feb 28, 2016
@filipw
Copy link
Member

filipw commented Feb 28, 2016

👍

@glennblock
Copy link
Contributor Author

@filipw while you are on a roll, how about checking this? #1129

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