Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Move Exception handling to General section #449

Closed
ssarmokadam opened this issue Aug 26, 2021 · 3 comments
Closed

Move Exception handling to General section #449

ssarmokadam opened this issue Aug 26, 2021 · 3 comments
Labels
invalid This doesn't seem right
Milestone

Comments

@ssarmokadam
Copy link
Member

Validate if current exception handling document can be followed for Quarkus as well or any further changes required in it. If it is common for both move it to general section else add Exception Handling for devon4j and Quarkus in respective sections.

@hohwille
Copy link
Member

The documentation is not spring specific in any way:
https://github.com/devonfw/devon4j/blob/master/documentation/guide-exceptions.asciidoc

However, it is specific for the requirement of i18n and requires the usage of mmm-util-nls.
As we are changing towards less framework and minimized dependencies as well as opt-in rather than opt-out, we should instead consider to strip all these mmm dependencies and put a minimal exception base-class into our app template that has no external dependencies.
BTW: If someone wants to use such library, he should consider using io.github.m-m-m:mmm-base instead. It is improved and much more lightweight. Then you can add io.github.m-m-m:mmm-nls on top to add I18N/L10N.

@hohwille
Copy link
Member

I created #562 and hence close this issue.

@hohwille
Copy link
Member

hohwille commented Jun 30, 2022

If it is common for both move it to general section else ...

This has always been the case:
https://github.com/devonfw/devon4j/blob/master/documentation/devon4j.asciidoc

@hohwille hohwille added the invalid This doesn't seem right label Jun 30, 2022
@hohwille hohwille added this to the rejected milestone Jun 30, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
invalid This doesn't seem right
Projects
None yet
Development

No branches or pull requests

2 participants