Skip to content

Add warning as error to release build Gitlab CI jobs - #1918

Open
bmhan12 wants to merge 8 commits into
developfrom
feature/han12/more_werror
Open

Add warning as error to release build Gitlab CI jobs#1918
bmhan12 wants to merge 8 commits into
developfrom
feature/han12/more_werror

Conversation

@bmhan12

@bmhan12 bmhan12 commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

This PR:

  • Adds warning as errors to release build type jobs for dane on Gitlab CI
  • Code changes (mainly mint) to eliminate warnings.

@bmhan12 bmhan12 added Testing Issues related to testing Axom CI Issues related to continuous integration labels Jul 15, 2026
Comment on lines -424 to -425
// Note: gcc@10.3.1 emits a '-Warray-bounds' warning in callers of this function
// about the sizes of nodeIds and coords not matching due to the runtime switch on dimension

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

This is the primary issue these mint changes are resolving.

Comment thread .gitlab/build_dane.yml

@Arlie-Capps Arlie-Capps left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thank you for these fixes, @bmhan12

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

Labels

CI Issues related to continuous integration Testing Issues related to testing Axom

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants