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

Minor proposals #17

Merged
merged 3 commits into from Oct 4, 2015
Merged

Minor proposals #17

merged 3 commits into from Oct 4, 2015

Conversation

nulltoken
Copy link
Contributor

  • Add a link to the NuGet package in the README
  • Update NuGet dependencies

@nulltoken
Copy link
Contributor Author

Wow! I think the Travis build log is really worth a shot 🎢 /cc @Therzok

I'm slightly puzzled by the AppVeyor issue.. Any idea?

@nulltoken
Copy link
Contributor Author

OK. Everything seems to be tamed now

@ethomson
Copy link
Collaborator

ethomson commented Oct 4, 2015

Neat! Thanks!

ethomson added a commit that referenced this pull request Oct 4, 2015
@ethomson ethomson merged commit 71040f3 into jasminsehic:master Oct 4, 2015
@Therzok
Copy link

Therzok commented Oct 4, 2015

The travis bot is using a really old mono version (3.12), so a bug report might be handy. I guess that shouldn't happen with a newer mono, but it's worth opening a bug report.

Without a solid repro or a test case, I'm not sure how to get this fixed.

The constructor does do a null check before, so it's probably a crash in the clone implementation.

@Therzok
Copy link

Therzok commented Oct 4, 2015

@nulltoken nulltoken deleted the ntk/nuget branch July 14, 2017 19:28
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

3 participants