Skip to content

Address feedback on JSON extraction for citation metadata#239

Merged
jordanpadams merged 2 commits intoi237from
copilot/sub-pr-238
Jan 7, 2026
Merged

Address feedback on JSON extraction for citation metadata#239
jordanpadams merged 2 commits intoi237from
copilot/sub-pr-238

Conversation

Copy link
Contributor

Copilot AI commented Jan 7, 2026

  • Add Logger import to XMLExtractor.java
  • Initialize Logger instance in XMLExtractor class
  • Replace System.err.println with Logger.warning()
  • Build and test the changes
  • Run code review
  • Run security scan

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

- Add java.util.logging.Logger import
- Initialize static final Logger instance for XMLExtractor class
- Replace System.err.println with log.warning() for error logging

This addresses the code review feedback to use a proper logging framework
instead of System.err for consistent error handling and better log management.

Co-authored-by: jordanpadams <33492486+jordanpadams@users.noreply.github.com>
@sonarqubecloud
Copy link

sonarqubecloud bot commented Jan 7, 2026

Copilot AI requested a review from jordanpadams January 7, 2026 00:38
Copilot stopped work on behalf of jordanpadams due to an error January 7, 2026 00:38
@jordanpadams jordanpadams changed the title [WIP] Address feedback on JSON extraction for citation metadata Address feedback on JSON extraction for citation metadata Jan 7, 2026
@jordanpadams jordanpadams marked this pull request as ready for review January 7, 2026 00:38
@jordanpadams jordanpadams requested a review from a team as a code owner January 7, 2026 00:38
@jordanpadams jordanpadams merged commit 0395a03 into i237 Jan 7, 2026
2 checks passed
@jordanpadams jordanpadams deleted the copilot/sub-pr-238 branch January 7, 2026 00:41
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