Tests cleanup by gsscoder · Pull Request #560 · commandlineparser/commandline
Navigation Menu
{{ message }}
commandlineparser / commandline Public
- Notifications You must be signed in to change notification settings
- Fork 481
Merged
moh-hassan merged 2 commits intocommandlineparser:developfrom
Jan 8, 2020Merged
Tests cleanup#560
moh-hassan merged 2 commits intocommandlineparser:developfrom
Tests cleanup#560
moh-hassan merged 2 commits intocommandlineparser:developfrom
Conversation
Copy link Copy Markdown
Contributor
gsscoder
commented
Jan 8, 2020
gsscoder
commented
Just some few order in test code.
moh-hassan reviewed Jan 8, 2020
| using FluentAssertions; | ||
| using Microsoft.FSharp.Core; | ||
| using CommandLine.Tests.Fakes; | ||
|
|
Copy link Copy Markdown
Collaborator
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
//Microsoft.FSharp.Core is needed for fsharp Target_Build. This is the reason for appveyor fail
Suggested change
| #if !SKIP_FSHARP | |
| using Microsoft.FSharp.Core; | |
| #endif |
moh-hassan
changed the base branch from
master
to
develop
moh-hassan
force-pushed
the
develop
branch
from
1439994 to
365542d
Compare
moh-hassan added 2 commits
January 9, 2020 00:07
moh-hassan
merged commit
f1a6527
into
commandlineparser:develop
gsscoder
deleted the
tests-cleanup
branch
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
2 participants