Add support for specifying Author and Committer (issue #2199)

[pvillard31]

Description

This solves #2199. See issue for more details.

Before submitting a PR:

• Changes must not break binary backwards compatibility. If you are unclear on how to make the change you think is needed while maintaining backward compatibility, CONTRIBUTING.md for details.
• Add JavaDocs and other comments explaining the behavior.
• When adding or updating methods that fetch entities, add @link JavaDoc entries to the relevant documentation on https://docs.github.com/en/rest .
• Add tests that cover any added or changed code. This generally requires capturing snapshot test data. See CONTRIBUTING.md for details.
• Run mvn -D enable-ci clean install site "-Dsurefire.argLine=--add-opens java.base/java.net=ALL-UNNAMED" locally. If this command doesn't succeed, your change will not pass CI.
• Push your changes to a branch other than main. You will create your PR from that branch.

When creating a PR:

• Fill in the "Description" above with clear summary of the changes. This includes:
  • If this PR fixes one or more issues, include "Fixes #" lines for each issue.
  • Provide links to relevant documentation on https://docs.github.com/en/rest where possible. If not including links, explain why not.
• All lines of new code should be covered by tests as reported by code coverage. Any lines that are not covered must have PR comments explaining why they cannot be covered. For example, "Reaching this particular exception is hard and is not a particular common scenario."
• Enable "Allow edits from maintainers".

[codecov]

Codecov Report

❌ Patch coverage is 92.77108% with 6 lines in your changes missing coverage. Please review.
✅ Project coverage is 85.12%. Comparing base (9152b37) to head (cacf7bb).

Files with missing lines	Patch %	Lines
...main/java/org/kohsuke/github/GHContentBuilder.java	84.61%	2 Missing ⚠️
...main/java/org/kohsuke/github/GHContentDeleter.java	93.75%	2 Missing ⚠️
...main/java/org/kohsuke/github/GHContentUpdater.java	94.44%	2 Missing ⚠️

Additional details and impacted files

@@             Coverage Diff              @@
##               main    #2200      +/-   ##
============================================
+ Coverage     85.03%   85.12%   +0.08%     
- Complexity     2499     2523      +24     
============================================
  Files           237      239       +2     
  Lines          7366     7449      +83     
  Branches        388      391       +3     
============================================
+ Hits           6264     6341      +77     
- Misses          871      877       +6     
  Partials        231      231              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 85.31%. Comparing base (3ecd46e) to head (f63133e).

Additional details and impacted files

@@             Coverage Diff              @@
##               main    #2200      +/-   ##
============================================
+ Coverage     85.14%   85.31%   +0.16%     
- Complexity     2525     2555      +30     
============================================
  Files           239      241       +2     
  Lines          7461     7544      +83     
  Branches        393      396       +3     
============================================
+ Hits           6353     6436      +83     
  Misses          872      872              
  Partials        236      236              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[pvillard31]

Hey @bitwiseman - do you think there is a chance to see a rc6 soon with this change? Asking to see if I can afford to wait for it, or if I should work around it with direct API calls in the meantime. Having this feature would be extremely useful for my usage.

[bitwiseman]

This isn't actually testing both signatures of each method.
I think you need a test that covers the with date methods and another that covers the non-date methods.

[bitwiseman]

Same comment here.