yash-agarwa-l opened a new pull request, #3436:
URL: https://github.com/apache/fory/pull/3436
## Why?
The current Dart exception hierarchy spreads runtime exceptions across four
separate files under `dart/packages/fory/lib/src/exception/`, which is overly
Java-style for idiomatic Dart.
## What needs to be done?
- Merge all runtime exception classes from:
- `exception/fory_exception.dart` (base `ForyException`)
- `exception/deserialization_exception.dart` (7 classes)
- `exception/serialization_exception.dart` (4 classes)
- `exception/registration_exception.dart` (6 classes)
into a single file: `dart/packages/fory/lib/src/fory_exception.dart`
- Update all imports across serializers, resolvers, context, and codegen
base to point to the new file.
- Delete the `exception/` subdirectory.
- Codegen exceptions (`codegen/exception/`) remain in their own directory
(separate concern).
## Does this introduce any user-facing change?
- [x] Does this introduce any public API change? (import paths change)
- [ ] Does this introduce any binary protocol compatibility change?
## Related
- Requested in #3434 review:
https://github.com/apache/fory/pull/3434#discussion_r2867271702
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
