Skip to content

Add job to build Windows executables to deploy.yml#2636

Merged
deslaughter merged 35 commits intoOpenFAST:devfrom
deslaughter:f/build-windows
Feb 14, 2025
Merged

Add job to build Windows executables to deploy.yml#2636
deslaughter merged 35 commits intoOpenFAST:devfrom
deslaughter:f/build-windows

Conversation

@deslaughter
Copy link
Collaborator

This PR is ready merge.

Feature or improvement description

This PR adds a job to the GitHub CI (deploy.yml) which will automatically build the full suite of Windows executables from the vs-build directory on release. This workflow can also be run manually which will allow for spot checking of the Visual Studio build without needing to build locally.

Impacted areas of the software

deploy.yml GitHub CI

Additional supporting information

This build uses the ifort compiler and Visual Studio. Once the ifx compiler is fully vetted, the build can switch to it. Until then, the build output will show warnings about the depreciation of ifort.

Copy link
Collaborator

@andrew-platt andrew-platt left a comment

Choose a reason for hiding this comment

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

this is fantastic! It will really speed up our builds at release.

Copy link
Collaborator

@andrew-platt andrew-platt left a comment

Choose a reason for hiding this comment

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

Looks good enough for now :) Let's see what it does on the next release.

@deslaughter deslaughter merged commit e85ee49 into OpenFAST:dev Feb 14, 2025
22 checks passed
@deslaughter deslaughter deleted the f/build-windows branch February 14, 2025 20:32
@andrew-platt andrew-platt mentioned this pull request Apr 28, 2025
35 tasks
@andrew-platt andrew-platt mentioned this pull request Jun 18, 2025
37 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments