zazuko / kopflos

kopflos - Linked Data APIs
MIT License
14 stars 6 forks source link

feat: express plugin, deploy plugin #178

Closed tpluscode closed 1 month ago

tpluscode commented 1 month ago

Introduces plugins feature with two plugins

@kopflos-cms/plugin-deploy-resources (separate package)

Will use talos to deploy a set of paths to the store using the default endpoint

@kopflos-cms/express/middleware (included in the express package)

Let's users register additional middlewares to run before and after kopflos handler

changeset-bot[bot] commented 1 month ago

🦋 Changeset detected

Latest commit: ed0d222e00846526b948f6fcf1df15dd6a7e13aa

The changes in this PR will be included in the next version bump.

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

codecov[bot] commented 1 month ago

Codecov Report

Attention: Patch coverage is 93.72385% with 15 lines in your changes missing coverage. Please review.

Project coverage is 92.90%. Comparing base (cca64b6) to head (ed0d222). Report is 25 commits behind head on cli.

Files with missing lines Patch % Lines
packages/cli/index.ts 0.00% 10 Missing :warning:
packages/core/lib/Kopflos.ts 92.18% 5 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## cli #178 +/- ## ========================================== + Coverage 88.60% 92.90% +4.30% ========================================== Files 22 25 +3 Lines 886 1043 +157 Branches 86 114 +28 ========================================== + Hits 785 969 +184 + Misses 99 73 -26 + Partials 2 1 -1 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

tpluscode commented 1 month ago

I just have a feeling that we're overdoing it with flexibility

I'm happy to discuss now. At which point do you think I'm taking the flexibility too far?