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

Update Javadoc @returns that have deviated from code #1463

Merged
merged 2 commits into from
Nov 16, 2024

Conversation

liblit
Copy link
Contributor

@liblit liblit commented Nov 16, 2024

We shouldn't describe a method as returning a "set of" anything when it actually returns an array.

We shouldn't describe a method as returning true or false when it actually returns a byte.

We shouldn't describe a method as returning a "set of" anything when it
actually returns an array.

We shouldn't describe a method as returning `true` or `false` when it
actually returns a `byte`.
@liblit liblit added the cleanup API cleanup and refactoring label Nov 16, 2024
@liblit liblit requested a review from msridhar November 16, 2024 19:44
@liblit liblit self-assigned this Nov 16, 2024
@liblit liblit enabled auto-merge (rebase) November 16, 2024 19:44
Copy link

Test Results

  813 files  ±0    813 suites  ±0   4h 3m 52s ⏱️ + 6m 2s
  735 tests ±0    718 ✅ ±0   17 💤 ±0  0 ❌ ±0 
5 029 runs  ±0  4 908 ✅ ±0  121 💤 ±0  0 ❌ ±0 

Results for commit 2c16c62. ± Comparison against base commit 6909f8e.

@liblit liblit merged commit 082ca09 into wala:master Nov 16, 2024
10 checks passed
@liblit liblit deleted the update-javadoc-returns branch November 16, 2024 20:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cleanup API cleanup and refactoring
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants