Skip to content

probably redundant jaxb - #562

Merged
denis-yuen merged 1 commit into
developfrom
feature/remove_jaxb
Jul 29, 2026
Merged

probably redundant jaxb#562
denis-yuen merged 1 commit into
developfrom
feature/remove_jaxb

Conversation

@denis-yuen

@denis-yuen denis-yuen commented Jul 27, 2026

Copy link
Copy Markdown
Member

Description
jaxb probably redundant with all the pruning back of the toolbackup module for virus scanning descriptors
Removing jaxb since it should be replaced by jakarta if anything

Review Instructions
Build and tests pass

Issue
https://ucsc-cgl.atlassian.net/browse/SEAB-6047

Security
None

Please make sure that you've checked the following before submitting your pull request. Thanks!

  • Check that you pass the basic style checks and unit tests by running mvn clean install in the project that you have modified (until https://ucsc-cgl.atlassian.net/browse/SEAB-5300 adds multi-module support properly)
  • Ensure that the PR targets the correct branch. Check the milestone or fix version of the ticket.
  • If you are changing dependencies, check with dependabot to ensure you are not introducing new high/critical vulnerabilities
  • If this PR is for a user-facing feature, create and link a documentation ticket for this feature (usually in the same milestone as the linked issue). Style points if you create a documentation PR directly and link that instead.

@denis-yuen denis-yuen self-assigned this Jul 27, 2026
@sonarqubecloud

Copy link
Copy Markdown

@codecov

codecov Bot commented Jul 27, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 15.85%. Comparing base (0ed81a6) to head (99681cf).

Additional details and impacted files
@@            Coverage Diff             @@
##             develop     #562   +/-   ##
==========================================
  Coverage      15.85%   15.85%           
  Complexity       100      100           
==========================================
  Files             40       40           
  Lines           2328     2328           
  Branches         186      186           
==========================================
  Hits             369      369           
  Misses          1934     1934           
  Partials          25       25           
Flag Coverage Δ
metricsaggregator 10.52% <ø> (ø)
toolbackup 15.85% <ø> (ø)
tooltester 10.52% <ø> (ø)
topicgenerator 10.52% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 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.

@denis-yuen
denis-yuen marked this pull request as ready for review July 27, 2026 18:39
@denis-yuen
denis-yuen requested review from a team, nolunwa-ucsc and svonworl and removed request for a team July 27, 2026 18:39
@denis-yuen

Copy link
Copy Markdown
Member Author

Can wait for 1.21

@denis-yuen
denis-yuen merged commit e3ffcae into develop Jul 29, 2026
14 checks passed
@denis-yuen
denis-yuen deleted the feature/remove_jaxb branch July 29, 2026 17:29
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