-
Notifications
You must be signed in to change notification settings - Fork 18
Package Naming #32
Comments
Yup, I like I'd perhaps keep the CLI separated from the rest, i.e. the
...and
...but then I'd also rename this repo so it is |
May be worth bringing up on brainstorm / TLS / @w-vi since I think this would be good to unify across the board. |
Do we have to support the previous names for backward compatibility? I mean, do we still publish to previous package names. |
…-0.20.0 Update fury-adapter-swagger to the latest version 🚀
I'd make a final release with old names and then deprecate the packages encouraging users to migrate to the newer ones. We should release a near-identical version under both names so people can move to the newer named packages without having to worry about other breaking changes to handle at the same time. Thus, we should get the current minim/Fury release completed before we rename the packages. |
@kylef This was wrongly closed because of moving OAS3 I think. |
Fury Adapters will now be in the form
Fury, the remote adapter and cli:
|
It would also be nice to be able to get |
@opichals that's a good point, we'll figure out those details. Initially I will bump the name and keep same CLI binary (so migration is simpler, then issue breaking changes separately such as renaming CLI binary etc). |
That might possibly be accomplished without a breaking change by introducing additional binary (same content) next to the existing one? |
Now that fury-cli is renamed to cli there are some broken pointers: |
We should settle on the package naming going forward along with how we will package adapters up as we can also use NPM Scoped packages https://docs.npmjs.com/misc/scope (
@apielements/openapi
etc). We have the following packages at the moment:minim-api-description
minim-parse-result
fury
fury-cli
fury-adapter-apib-parser
fury-adapter-apiary-blueprint-parser
fury-adapter-swagger
swagger-zoo
fury-adapter-apib-serializer
Perhaps the adapters could become something like
@apielements/openapi2-parser
,@apielements/openapi3-parser
,@apielements/apiblueprint-parser
. We could offer@apielements/cli
and some package containing the base (fury.js) and namespace.We could also offer a root level package
apielements
which provides everything, it could provide all of the adapters that we provide along side the CLI tool. That way consumers like Dredd (@honzajavorek), and Documentation (@char0n) have a single dependency useapielements
which has all of the provided adapters already included without them having to deal with each adapter.I know @honzajavorek had some gripes with using a dash in
api-elements
package so we can settle that too. I'd appreciate any feedback on the naming convention here./c @pksunkara
The text was updated successfully, but these errors were encountered: