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

[Hotfix] Update Main.cs and ScriptCompiler.cs for MONO5 #3250

Closed
wants to merge 2 commits into from
Closed

[Hotfix] Update Main.cs and ScriptCompiler.cs for MONO5 #3250

wants to merge 2 commits into from

Conversation

Xeroxxx
Copy link

@Xeroxxx Xeroxxx commented Mar 26, 2018

Updated for compatibility with MONO5.
Checked with Mono 3.2.8 and Mono 5.10.0

Changes in Main.cs
+ Checking for Mono Version #if Mono and Code is a double check right now
Changes in ScriptCompiler.cs
+ Changed file encoding (^M) to be inline with others
+ Changed lines 233 to 236 to resolve errors with Mono 5

Updated for compatibility with MONO5.

Checked with Mono 3.2.8 and Mono 5.10.0
corrected indents
@Xeroxxx
Copy link
Author

Xeroxxx commented Mar 27, 2018

@Xeroxxx Xeroxxx changed the title Update Main.cs and ScriptCompiler.cs for MONO5 [Hotfix] Update Main.cs and ScriptCompiler.cs for MONO5 Mar 28, 2018
@Xeroxxx
Copy link
Author

Xeroxxx commented Apr 1, 2018

Nice Hotfix if it takes 2 weeks to review someone 5 lines of code.

@Voxpire
Copy link
Contributor

Voxpire commented Apr 2, 2018

It's not just 5 lines of code, you changed the formatting of ScriptCompiler.cs which makes it hard to discern what has been changed in a side-by-side difference. In future, commit the changes, then commit file formatting changes separately.

Copy link
Contributor

@Voxpire Voxpire left a comment

Choose a reason for hiding this comment

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

Commit changes with clear annotations to differentiate them from file formatting changes.

The Core edit for displaying the environment - there's no point in using a formatted string inline with concatenation, pick one or the other.

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.

None yet

2 participants