Skip to content

feat:java 21 modernisation#165

Merged
sr4850 merged 3 commits into
mainfrom
refactor/java-21-modernisation
Apr 14, 2026
Merged

feat:java 21 modernisation#165
sr4850 merged 3 commits into
mainfrom
refactor/java-21-modernisation

Conversation

@sr4850

@sr4850 sr4850 commented Apr 14, 2026

Copy link
Copy Markdown
Collaborator

Description

Related issue: JIRA_TICKET_NUMBER

Before submitting (or marking as "ready for review")

  • Does the pull request title follow the conventional commit specification?
  • Have you performed a self-review of the code
  • Have you have added tests that prove the fix or feature is effective and working
  • Did you make sure to update any documentation relating to this change?

sr4850 and others added 3 commits April 14, 2026 16:22
- Remove redundant private keyword from enum constructors
- Standardise enum field naming to 'value' across all enums
- Replace switch statement with data-driven approach in TrafficArea
- Use ThreadLocalRandom instead of shared Random for thread safety
- Replace String.format() with .formatted() in TrafficArea
- Deprecate trafficAreaList() and enforcementAreaList() (use values())
- Remove @NotNull annotations from getEnum() in favour of null-safe streams
- Update README to state Java 21 requirement

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
- Add TrackingStatus enum matching olcs-transfer valid values (0-3)
- Add withAllStatuses() convenience method to TrackingBuilder
- Fix duplicate taxiPhvStatus in @JsonPropertyOrder
- Fix discsStatus/vehiclesPsvStatus types from Object to String
- Add missing vehiclesDeclarationsStatus field (present in olcs-transfer)
- Remove redundant @JsonProperty from getters/setters (field-level sufficient)
- Use TrackingStatus.ACCEPTED in GrantLicence.createOverview()
- Simplify createOverview() from 29-line chain to single withAllStatuses() call

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Addresses Dependabot security alert #8.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@sr4850 sr4850 merged commit 78d010e into main Apr 14, 2026
5 checks passed
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