Skip to content

Remove excess util folder#1072

Merged
andchiind merged 2 commits intoequinor:mainfrom
andchiind:949-remove-utils
Jan 26, 2026
Merged

Remove excess util folder#1072
andchiind merged 2 commits intoequinor:mainfrom
andchiind:949-remove-utils

Conversation

@andchiind
Copy link
Contributor

@andchiind andchiind commented Jan 22, 2026

Closes #949

Breaking change since some ISAR implementations import the JSON encoder.

Ready for review checklist:

  • A self-review has been performed
  • All commits run individually
  • Temporary changes have been removed, like logging, TODO, etc.
  • The PR has been tested locally
  • A test has been written
    • This change doesn't need a new test
  • Relevant issues are linked
  • Remaining work is documented in issues
    • There is no remaining work from this PR that requires new issues
  • The changes do not introduce dead code as unused imports, functions etc.

@andchiind andchiind self-assigned this Jan 22, 2026
@andchiind andchiind added breaking-change A breaking change which introduces changes to the public APIs improvement Improvement to existing functionality labels Jan 22, 2026
Copy link
Contributor

@Christdej Christdej left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@andchiind andchiind merged commit 7c4c422 into equinor:main Jan 26, 2026
7 checks passed
@andchiind andchiind deleted the 949-remove-utils branch January 26, 2026 10:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

breaking-change A breaking change which introduces changes to the public APIs improvement Improvement to existing functionality

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remove the EnhancedJSONEncoder

2 participants