-
Notifications
You must be signed in to change notification settings - Fork 4
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
Rename repository to reflect wider scope than just observer API #6
Comments
The most I like is |
why not simply use different repos? |
@yurishkuro Was thinking that the filter/interceptor API would also use the |
It's it worth creating intermodule dependencies because of one pojo? |
The APIs should be in a single module, but the wrapper discussed in #5 would probably be better in a separate module. |
@pavolloffay @felixbarny @yurishkuro As part of this issue, would like to do the following:
Any objections? |
As discussed in this thread (#3 (comment)) we should rename this repo to something more appropriate to cover observer and filter API.
java-patterns
- too genericjava-tracer-decorator
andjava-tracer-wrapper
- I think these are too implementation specific, and focus just on the wrapping/decorator rather than on the APIsjava-api-extensions
andjava-tracer-extensions
- these are in the right direction, wonder if a truncated form would work - e.g.java-ext
The text was updated successfully, but these errors were encountered: