herbsjs / herbs-cli

Herbs CLI
https://herbsjs.org/
MIT License
29 stars 30 forks source link

Updated version of herbsjs and glues #188

Closed vitorgamer58 closed 1 year ago

vitorgamer58 commented 1 year ago

The use of ctx.data instead of ctx.user makes a project created with herbs new not have problems with the user feature inside ctx.user implemented in version 2.0.0 of buchu. Refer to: https://github.com/herbsjs/buchu/pull/85

Proposed Changes

  1. Update herbs to ^2.0.0
  2. Update herbarium to ^1.5.0
  3. Update herbsshelf to ^5.0.1
  4. Update herbs2mongo to ^4.0.0
  5. Update herbs2knex to ^2.1.1
  6. Update herbs2gql to ^3.0.0
  7. Update herbs2rest to ^4.1.1
  8. Usecase template should use ctx.data

Readiness Checklist

Author/Contributor

Reviewing Maintainer

codecov[bot] commented 1 year ago

Codecov Report

Merging #188 (870b1f5) into main (d50fbb8) will not change coverage. The diff coverage is n/a.

:exclamation: Current head 870b1f5 differs from pull request most recent head 58f8f85. Consider uploading reports for the commit 58f8f85 to get more accurate results

@@           Coverage Diff           @@
##             main     #188   +/-   ##
=======================================
  Coverage   68.75%   68.75%           
=======================================
  Files          28       28           
  Lines         560      560           
=======================================
  Hits          385      385           
  Misses        175      175           
Impacted Files Coverage Δ
src/generators/src/packagejson.js 100.00% <ø> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

vitorgamer58 commented 1 year ago

Working fine with new version 2.0.0 of herbs. Tested: Add a user, get the list of all users, get by id, edit a user and delete by id (all usecases). No break in generated unit tests. Screenshot_547 Screenshot_548