search

cloudstark / quarkus-zalando-problem-extension

Quarkus extension for application/problem+json (RFC7807)
Apache License 2.0
4 stars 5 forks source link

Rename extension to quarkus-zalando-problem-extension #8

Closed SchulteMarkus closed 4 years ago

SchulteMarkus commented 4 years ago

In my opinion the name of this extension is to reduced, I think a more specific quarkus-zalando-problem-extension would be better. Background: Microprofile itself seems to have discovered, that it would be a good idea to implement RFC-7807, see https://github.com/eclipse/microprofile-sandbox/tree/master/proposals/problem-details. I can imagine there will be a "Microprofile problem" or "Quarkus problem" in the future. If this happens, "quarkus-problem-extension" can be confusing, I think.

CrystalMethod commented 4 years ago

This repository started as a PoC to learn how to write Quarkus extensions. So this current name was obvious and not so wrong. Thanks for the link - I didn't know about these efforts. Under these circumstances I can agree to the name change. The impact should be minimal, since there are no public artifacts yet.

SchulteMarkus commented 4 years ago

Finding a good name ;-)

CrystalMethod commented 4 years ago

I'd suggest to change the repository name as well - not just the artifacts (#9).

SchulteMarkus commented 4 years ago

And maybe change the feature name in ProblemProcessor, too.

SchulteMarkus commented 4 years ago

And the README is outdated right now

CrystalMethod commented 4 years ago

The lack of CI is noticeable..at least for compilation errors and the like.. ;-) This will certainly be one of my next tasks

CrystalMethod commented 4 years ago

The README can certainly be further improved. But the renaming of the artifacts and the repository is completed.