Skip to content

Conversation

@dzenanz
Copy link
Member

@dzenanz dzenanz commented May 30, 2024

Closes #138.

@dzenanz dzenanz marked this pull request as ready for review May 30, 2024 17:52
Sample failure output:

2: Test command: C:\Misc\SlicerExecutionModel\bld22\ModuleDescriptionParser\bin\Debug\Parser1Test.exe "C:/Misc/SlicerExecutionModel/ModuleDescriptionParser/Testing/TestData/ParserTest2.xml"
2: Working Directory: C:/Misc/SlicerExecutionModel/bld22/ModuleDescriptionParser/Testing
2: Test timeout computed to be: 1500
2: not well-formed (invalid token) at line 111
1/2 Test #2: Parser1Test2 .....................***Failed    0.02 sec
@dzenanz dzenanz requested review from aylward and sjh26 May 30, 2024 18:11
Copy link
Contributor

@sjh26 sjh26 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm!

@dzenanz
Copy link
Member Author

dzenanz commented May 30, 2024

The CI seems to be broken. The tests succeed on both my local Linux and Windows machines. Should we merge this, or wait for someone to fix the CI and rebase?

Copy link
Member

@aylward aylward left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice!

Failing test seems to be an authentication issue (requesting write access to this repo during testing - weird). I've tested this on my laptop: Windows and Linux. Works great on my custom (non-Slicer) apps that use SlicerExecutionModel for CLI parsing.

@dzenanz dzenanz merged commit 4325526 into master May 31, 2024
@dzenanz dzenanz deleted the sharedForward branch May 31, 2024 20:04
@hjmjohnson
Copy link
Member

@dzenanz THANK YOU!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

KWSys / itksys no longer has a SharedForward.h.in file

4 participants