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

feat(build): add -Wextra to "picky" compiler flags #588

Merged
1 commit merged into from
Sep 27, 2024

Conversation

@ghost ghost requested a review from bwbarrett as a code owner September 11, 2024 21:25
@ghost ghost force-pushed the aws-nslick/stack/29 branch from 487cce2 to 2108f5f Compare September 11, 2024 21:26
@ghost ghost force-pushed the aws-nslick/stack/30 branch from ae9e42c to 8d39709 Compare September 11, 2024 21:26
@ghost ghost changed the base branch from aws-nslick/stack/29 to master September 11, 2024 23:12
@ghost ghost requested review from rajachan, rauteric and a team as code owners September 11, 2024 23:12
@ghost ghost force-pushed the aws-nslick/stack/30 branch 2 times, most recently from 163c6bf to a7cca1a Compare September 11, 2024 23:12
@ghost ghost force-pushed the aws-nslick/stack/30 branch 2 times, most recently from decf723 to 93a7011 Compare September 12, 2024 00:57
@ghost ghost changed the base branch from master to aws-nslick/stack/36 September 12, 2024 00:58
@ghost ghost changed the base branch from aws-nslick/stack/36 to master September 12, 2024 01:09
@ghost ghost force-pushed the aws-nslick/stack/30 branch from 93a7011 to c51d55a Compare September 12, 2024 01:10
@ghost ghost changed the base branch from master to aws-nslick/stack/36 September 12, 2024 01:11
@ghost ghost changed the base branch from aws-nslick/stack/36 to master September 12, 2024 01:51
@ghost ghost force-pushed the aws-nslick/stack/30 branch from c51d55a to abbf61b Compare September 12, 2024 01:51
@ghost ghost changed the base branch from master to aws-nslick/stack/36 September 12, 2024 01:52
@ghost ghost changed the base branch from aws-nslick/stack/36 to master September 13, 2024 17:49
@ghost ghost force-pushed the aws-nslick/stack/30 branch from abbf61b to ec96367 Compare September 13, 2024 17:49
@ghost ghost changed the base branch from master to aws-nslick/stack/36 September 13, 2024 17:50
rauteric
rauteric previously approved these changes Sep 13, 2024
@ghost ghost changed the base branch from aws-nslick/stack/36 to master September 14, 2024 22:36
@ghost ghost dismissed rauteric’s stale review September 14, 2024 22:36

The base branch was changed.

@ghost ghost force-pushed the aws-nslick/stack/30 branch from ec96367 to 03c4994 Compare September 14, 2024 22:37
@ghost ghost changed the base branch from master to aws-nslick/stack/36 September 14, 2024 22:38
Signed-off-by: Nicholas Sielicki <nslick@amazon.com>

stack-info: PR: #588, branch: aws-nslick/stack/30
@ghost ghost changed the base branch from aws-nslick/stack/36 to master September 17, 2024 20:25
@ghost ghost force-pushed the aws-nslick/stack/30 branch from 03c4994 to 53ceba2 Compare September 17, 2024 20:25
@ghost ghost changed the base branch from master to aws-nslick/stack/36 September 17, 2024 20:26
@ghost ghost merged commit c56ceb5 into aws-nslick/stack/36 Sep 27, 2024
32 of 33 checks passed
@ghost ghost mentioned this pull request Sep 27, 2024
@ghost ghost deleted the aws-nslick/stack/30 branch September 27, 2024 00:56
ghost pushed a commit that referenced this pull request Oct 1, 2024
Signed-off-by: Nicholas Sielicki <nslick@amazon.com>
aws-ofiwg-bot pushed a commit to aws-ofiwg-bot/aws-ofi-nccl that referenced this pull request Oct 4, 2024
Signed-off-by: Nicholas Sielicki <nslick@amazon.com>
This pull request was closed.
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.

2 participants