Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Comments

due to -Werror compiler arg, build is failing on warning#760

Open
TbirdDuncan wants to merge 1 commit intotemporalio:main from
TbirdDuncan:fix_werror
Open

due to -Werror compiler arg, build is failing on warning #760
TbirdDuncan wants to merge 1 commit intotemporalio:main from
TbirdDuncan:fix_werror

Conversation

@TbirdDuncan
Copy link

@TbirdDuncan TbirdDuncan commented Nov 18, 2025

What was changed

Added a comment to an ignored catch block in Starter.java. Builds fail as there is a defined compiler arg: -Werror

 compileJava {
 options.compilerArgs << "-Werror"
 }

Why?

So we can build

  1. How was this tested:
    built and tested

Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.


TristanAnderson seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

1 similar comment
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.


TristanAnderson seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

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

Reviewers

@tsurdilo tsurdilo Awaiting requested review from tsurdilo tsurdilo is a code owner

@antmendoza antmendoza Awaiting requested review from antmendoza antmendoza is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

AltStyle によって変換されたページ (->オリジナル) /