The module needs a composer.json file

It depends on the licenses of the used libraries and if you have to agree to something while downloading them or you are not allowed to spread them out from anohter than the original download source, that's why many module maintainers hasitate to put all in composer dependencies. But I agree with you that if we are forced to use composer in Drupal why do we do not use it consequently all over the place. I feel with you and agree on this attempt.

Feel free to research on the license of the respective dependencies and if it is allowed to re-share them etc. If all is fine you can try to work on a solution if you like.

Again: thanks for your report.

chetan 11 → made their first commit to this issue’s fork.

Please check the above MR.

First nit picks:

+    "require": {
+        "drupal/core": "8.* || 9.*"
+     }

2.x-dev requires Drupal 8 | 9 | 10
But review not complete yet.

Yes indeed, for that line rather,
"drupal/core": "^9 || ^10"
But how to test the composer.json locally? Since the one takes into account is the repository one. And there are further crucial needs.

Chris64 → changed the visibility of the branch 3415937-the-module-needs to hidden.

Chris64 → changed the visibility of the branch 3415937-the-module-needs to active.

The situation is there is two versions of the composer.json. The second one should be considered as a minimal required version. The first one is more developed. The plan is then,
- Evaluate the second one.
- Chose some elements from the first one to add to the second one.
That is to say creating and merging not done.

Maybe the last composer.json needs some explanations. The idea is to make here what is asked to a user in the README.md to make in the main composer.json file to install the datatables library with composer. The process has already been reviewed and tested in an older issue: 📌 Drupal core has changed jQuery once() [Drupal 10 compatibility] Fixed .

Other problem with the original version,
"license": "GPL-2.0+",
registered as Deprecated License Identifiers in https://spdx.org/licenses/.

About the license of the library, request from #4 📌 The module needs a composer.json file Needs review . One means to get it,
composer show datatables/datatables | grep license
giving,
license : MIT License (MIT) (OSI approved) https://spdx.org/licenses/MIT.html#licenseText
Verify the library version,
composer show datatables/datatables | grep version
To verify if the license is compatible with Drupal: https://spdx.org/licenses/. And an other information about the datatables library the archive #3410585: Packaging allowlist archive → . Search datatables in the page. Drupalorg_whitelist is self-explanatory.

liam morland → made their first commit to this issue’s fork.