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

bump max visual studio version to latest alpha #2171

Closed
wants to merge 10 commits into from

Conversation

zippo227
Copy link
Contributor

@zippo227 zippo227 commented Jun 2, 2017

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
Collaborator

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
Copy link

dnfclas commented Jun 2, 2017

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
Copy link
Contributor Author

zippo227 commented Jun 2, 2017

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
Copy link
Contributor

@spouliot spouliot left a comment

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
Copy link
Contributor Author

zippo227 commented Jun 3, 2017

@spouliot I understand and changed as requested

@zippo227
Copy link
Contributor Author

zippo227 commented Jun 3, 2017

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 (xamarin#2171)" (xamarin#2194)
* xamarin/maccore@9939a3812d [mlaunch] Accept the latest C# language version in projects (xamarin#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