Skip to content

bump max visual studio version to latest alpha - #2171

Closed
zippo227 wants to merge 10 commits into
dotnet:masterfrom
zippo227:master
Closed

bump max visual studio version to latest alpha#2171
zippo227 wants to merge 10 commits into
dotnet:masterfrom
zippo227:master

Conversation

@zippo227

@zippo227 zippo227 commented Jun 2, 2017

Copy link
Copy Markdown
Contributor

I increased the allowed visual studio version, and I added a fix that also prevents a null exception from attempting to be read during build error generation.

@monojenkins

Copy link
Copy Markdown
Contributor

Hello! I'm the build bot for the Mono project.

I need approval from a Mono team member to build this pull request. A team member should reply with "approve" to approve a build of this pull request, "whitelist" to whitelist this and all future pull requests from this contributor, or "build" to explicitly request a build, even if one has already been done.

Contributors can ignore this message.

@dnfclas

dnfclas commented Jun 2, 2017

Copy link
Copy Markdown

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot

…eption while throwing an exception during a build.
@zippo227

zippo227 commented Jun 2, 2017

Copy link
Copy Markdown
Contributor Author

You can see more about the issue that allow the bug to be seen here. https://bugzilla.xamarin.com/show_bug.cgi?id=57124

@zippo227 zippo227 changed the title bump max visual studio version to latest alpha bump max visual studio version to latest alpha and fix compilation error Jun 2, 2017

@spouliot spouliot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

That needs to be 2 separate PR as they are unrelated changes, e.g. One might have to be reverted without the other

@zippo227 zippo227 changed the title bump max visual studio version to latest alpha and fix compilation error bump max visual studio version to latest alpha Jun 3, 2017
@zippo227

zippo227 commented Jun 3, 2017

Copy link
Copy Markdown
Contributor Author

@spouliot I understand and changed as requested

@zippo227

zippo227 commented Jun 3, 2017

Copy link
Copy Markdown
Contributor Author

I think this got too bloated and it is not recognizing my rebase. I've made a new, cleaner pull request. #2173

@zippo227 zippo227 closed this Jun 3, 2017
@zippo227
zippo227 deleted the master branch June 3, 2017 17:40
dalexsoto pushed a commit to dalexsoto/xamarin-macios that referenced this pull request Apr 6, 2020
The name of the dll changed, ergo the prefix changed.

New commits in xamarin/maccore:

* xamarin/maccore@a14f74b40a Revert "[Localization] Rework mlaunch (dotnet#2171)" (dotnet#2194)
* xamarin/maccore@9939a3812d [mlaunch] Accept the latest C# language version in projects (dotnet#2193)

Diff: https://github.com/xamarin/maccore/compare/fdeec0f4ccdfdb9111d594b7f7189cd58e06216d..a14f74b40ac5ed01a9a384d758b50e5a7c563569
dalexsoto added a commit to dalexsoto/xamarin-macios that referenced this pull request Apr 6, 2020
dalexsoto added a commit that referenced this pull request Apr 6, 2020
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.

4 participants