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.

Sign up for GitHub

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

Jump to bottom

Remove Mono.Options from the TestRunner #72

Closed
mandel-macaque opened this issue Apr 23, 2020 · 3 comments
Closed

Remove Mono.Options from the TestRunner #72

mandel-macaque opened this issue Apr 23, 2020 · 3 comments
Labels
apple iOS/tvOS/WatchOS/Mac Catalyst area test-runners TestRunner area

Comments

@mandel-macaque
Copy link
Member

We want to keep the dependencies to the minimum.
@mandel-macaque mandel-macaque added test-runners TestRunner area apple iOS/tvOS/WatchOS/Mac Catalyst area labels Apr 23, 2020
@mandel-macaque mandel-macaque mentioned this issue Apr 23, 2020
Merged
@premun premun changed the title [iOS] Remove all occurrences of Mono.Options. [iOS] Remove Mono.Options from the TestRunner Apr 23, 2020
@premun
Copy link
Member

premun commented Apr 27, 2020

Mono.Options.dll is 45 kB, maybe we want to keep it after all? Doesn't look that bad and we would bring in another code that would have to do the same instead so..

@premun
Copy link
Member

premun commented Jun 5, 2020

@mandel I would just close this one? Mono.Options is tiny and dependency-less. The NUnit/xUnit runners are way bigger and this brings value to us actually.

@akoeplinger
Copy link
Member

Yeah I agree, using Mono.Options sounds fine.

@premun premun closed this as completed Jun 16, 2020
@premun premun changed the title [iOS] Remove Mono.Options from the TestRunner Remove Mono.Options from the TestRunner Aug 19, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
apple iOS/tvOS/WatchOS/Mac Catalyst area test-runners TestRunner area
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@akoeplinger @mandel-macaque @premun