Skip to content

Use Travis' csharp language setup #1300

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

Merged
merged 1 commit into from
Apr 15, 2016
Merged

Use Travis' csharp language setup #1300

merged 1 commit into from
Apr 15, 2016

Conversation

carlosmn
Copy link
Member

Use the configuration Travis already knows about to reduce how much
custom code we're running.

As it's now easy to do, test on both 3.12 and 4.2 versions of mono. This
also has the advantage of making us test on both systems with the same
version.

This fixes #1032, fixes #1082, supersedes #1086, supersedes #1129

@carlosmn carlosmn force-pushed the cmn/travis-csharp branch from 4dbb815 to 3ec31d6 Compare April 13, 2016 10:25
@carlosmn carlosmn changed the title [WIP] Use Travis' csharp language setup Use Travis' csharp language setup Apr 13, 2016
Use the configuration Travis already knows about to reduce how much
custom code we're running.

As it's now easy to do, test on both 3.12 and 4.2 versions of mono. This
also has the advantage of making us test on both systems with the same
version.
@carlosmn carlosmn force-pushed the cmn/travis-csharp branch from 3ec31d6 to 85ad800 Compare April 13, 2016 10:28
@carlosmn
Copy link
Member Author

This seems like it's working. We build with 3.12 so we support those who can't upgrade for whatever reason, but also with 4.2 so we build with something recent. It also unifies which versions we test on which OS, which was different between Ubuntu and OS X. @ethomson thoughts?

@ethomson ethomson merged commit cd8ca84 into master Apr 15, 2016
@ethomson
Copy link
Member

This is really delightful. Thanks for tackling this. And thanks @FeodorFitsner for making this so easy.

@bording bording deleted the cmn/travis-csharp branch April 21, 2019 02:00
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.

Leverage Travis csharp language Use LEAKS defines on Mono and use Mono 4.
2 participants