-
Notifications
You must be signed in to change notification settings - Fork 564
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Windows Github Action #7227
base: main
Are you sure you want to change the base?
Windows Github Action #7227
Conversation
inject/processor/src/main/java/io/helidon/inject/processor/ActiveProcessorUtils.java
Outdated
Show resolved
Hide resolved
.../test/java/io/helidon/integrations/oci/sdk/runtime/OciAuthenticationDetailsProviderTest.java
Outdated
Show resolved
Hide resolved
...o/helidon/nima/tests/integration/http2/webserver/Http2ErrorHandlingWithOutputStreamTest.java
Outdated
Show resolved
Hide resolved
...ebclient/webclient/src/test/java/io/helidon/nima/tests/integration/client/HttpProxyTest.java
Outdated
Show resolved
Hide resolved
c4c4558
to
50d7204
Compare
builder/tests/builder/src/test/java/io/helidon/builder/test/CustomNamedTest.java
Show resolved
Hide resolved
inject/processor/src/main/java/io/helidon/inject/processor/ActiveProcessorUtils.java
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In general, I can confirm these updates work for my Windows build system. I would recommend addressing the file-system separator, end-of-line and other issues (if they exist) using the system properties and appropriate settings. (e.d. system properties: line.separator, File.separator, and File.pathSeparator)
50d7204
to
3dcc933
Compare
c4c8fb8
to
5de8a6c
Compare
fc5656f
to
b958fd2
Compare
Description
Fixes #6717
Fixes #6714
Documentation
Build Helidon on windows as part of the pipeline
The build last 4 more minutes and reach 9 mins in total.