Skip to content

merge dev to master#881

Merged
jrusbatch merged 38 commits into
masterfrom
dev
Dec 24, 2014
Merged

merge dev to master#881
jrusbatch merged 38 commits into
masterfrom
dev

Conversation

@adamralph
Copy link
Copy Markdown
Contributor

filipw and others added 30 commits June 18, 2014 19:30
833 Allow log level to be set in debug mode.
Conflicts:
	src/ScriptCs.Engine.Roslyn/RoslynScriptEngine.cs
…orrect with the command dictionary (instead of list)
Added support for :references, :usings, :vars and :help in REPL
 - redundant code
 - line length
 - guard clauses
 - EOL at end of file
 - readonly fields
 - replace if with null coalesce
 - replace if with inline conditional
 - whitespace
@adamralph adamralph mentioned this pull request Dec 23, 2014
9 tasks
@filipw
Copy link
Copy Markdown
Member

filipw commented Dec 23, 2014

this is good to go imho

@adamralph
Copy link
Copy Markdown
Contributor Author

in mine too 😛

@filipw
Copy link
Copy Markdown
Member

filipw commented Dec 23, 2014

@scriptcs/core thoughts? comments? opinions?

@filipw
Copy link
Copy Markdown
Member

filipw commented Dec 23, 2014

btw, do we host the ZIP of latest release anywhere on github? we should be doing that for automated installations that cannot use chocolatey

@jrusbatch
Copy link
Copy Markdown
Member

This looks good to me. 👍

@filipw We hosted releases on Github due to the distribution constraints on Roslyn. Not sure if they've changed with any of the recent releases, though.

@jrusbatch jrusbatch self-assigned this Dec 24, 2014
jrusbatch added a commit that referenced this pull request Dec 24, 2014
@jrusbatch jrusbatch merged commit c2f2318 into master Dec 24, 2014
@glennblock
Copy link
Copy Markdown
Contributor

👍

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants