Rohan
Rohan
Hi @rickstaa, I would like to contribute. So if I understand correctly, the above mentioned imports should change to use ES6 syntax as below: `import I18n from "../common/I18n";` Also the...
@rickstaa Yes, thanks
@rickstaa, do you want me to change the imports/exports in all files as part of this issue or are there any specific files I need to exclude?
@rickstaa I am making changes to all the `*.test.js` files since they also contain CommonJS modules way of importing and exporting. Can you please confirm how to run the tests?
@rickstaa I have made the changes to use ES6 import/export. And I have tested it by running on my local as per the instructions in [CONTRIBUTING.md](https://github.com/anuraghazra/github-readme-stats/blob/master/CONTRIBUTING.md) and it works. Screenshots...
@rickstaa even if we do keep the test files on the CommonJS syntax, the test fails when it executes the ES import statements in the file being tested. Here is...
@rickstaa @Rudxain I have changed the CommonJS import/export to ES6 imports/exports and updated [https://github.com/rsk2/github-readme-stats/tree/feature/GRS-1955-change-commonjs-imports](https://github.com/rsk2/github-readme-stats/tree/feature/GRS-1955-change-commonjs-imports). The tests are still failing as discussed. Should I create a pull request?
@Rudxain OK, I have created a draft PR.
Hey @rickstaa, I tried to implement the solution that is provided in https://stackoverflow.com/a/69059786/8135687, however I am still getting some errors.
> @rsk2 Thanks for letting me know. I just tried fixing these errors myself (see https://github.com/anuraghazra/github-readme-stats/tree/change_to_es_imports), but I keep getting the following errors: > > ``` > SyntaxError: The requested...