Skip to content
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

February 2025 - Confusing text regarding cds.test dependencies #1692

Open
andrew-barnard opened this issue Mar 7, 2025 · 1 comment
Open

Comments

@andrew-barnard
Copy link

The documentation at https://cap.cloud.sap/docs/releases/feb25#cds-test states:

Get rid of the extra dependencies that you get now through @cap-js/cds-test
npm rm axios chai chai-subset chai-as-promised

The npm dependencies tab for @cap-js/cds-test lists the following dependencies, which are installed when @cap-js/cds-test is installed. See https://www.npmjs.com/package/@cap-js/cds-test?activeTab=dependencies

  • axios
  • chai
  • chai-as-promised
  • chai-subset

Because these are listed as dependencies, I found the instruction to get rid of them confusing. If after installation of @cap-js/cds-test these dependencies are not required, perhaps the text can make this more explicit?

I suggest the following text:

Get rid of the unnecessary dependencies that you get now through @cap-js/cds-test

@renejeglinsky
Copy link
Contributor

Thanks @andrew-barnard ,

I totally get your point. Should we even be more elaborate on that? Like:

This package comes with dependencies you had to maintain separately until now. So get rid of the unnecessary dependencies and only maintain the @cap-js/cds-test package going forward.

All the best,
René

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants