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.

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

Add CI builds for macOS, Windows, and Ubuntu #299

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

K20shores
Copy link

Name and Institution (Required)

Name: Kyle Shores
Institution: National Center for Atmospheric Research (NCAR)

Describe the update

This builds HEMCO on macOS, Ubuntu, and Windows. All platforms build with debug and release (why not?). Ubuntu and macOS also build both configurations with GCC 12, 13, and 14. Adding these files ensures HEMCO builds on all platforms. While the plans are still unclear, there has been talk of including HEMCO into MUSICA. One of our software requirements to be have software included in MUSICA is to build on all platforms.

Expected changes

None

Reference(s)

N/A

Related Github Issue

N/A

@lizziel lizziel added this to the 3.11.0 milestone Jan 10, 2025
@lizziel lizziel self-assigned this Jan 10, 2025
@yantosca yantosca self-requested a review January 27, 2025 20:22
Copy link
Contributor

@yantosca yantosca left a comment

Choose a reason for hiding this comment

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

Thanks for this PR @K20shores. Would you also be able to add a short note to the CHANGELOG.md file describing this update? Other than that, this looks good to merge and we can get it into the HEMCO 3.11.0 development stream.

@K20shores
Copy link
Author

@yantosca done! I think, please let me know if I didn't write the updates correctly

Copy link
Contributor

@yantosca yantosca 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 @K20shores! Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants