Skip to content

Update dependencies and clean up the repository #136

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

Conversation

surilindur
Copy link
Contributor

This is a set of changes that are all kind of connected to each other, and since only 19 files are changed, I decided to keep them in the same PR. The main changes are:

  • Bumping dependencies where appropriate, and also fixing the resulting errors from version updates.
  • Removing redundant files such as .DS_Store and debug that are not needed for anything and were likely included by accident.
  • Moving index.ts under lib/ to ensure all the exported stuff is now under lib/ and not at the root. This also helps simplify some rules.
  • Splitting Jest configs to their own file, as in the Comunica repository.
  • Adding --maxWorkers=50% to the test script, because it runs a lot faster that way and uses a lot less memory. I think Jest, by default, uses way too many workers for the tests in this repository. It was eating up around 20GB of memory for me when running the tests without that worker limit, and the tests took several times longer, as well.

Any feedback would be welcome! 🙂 I just ended up having a look at this repo after some warnings appeared in Comunica link traversal (that were unrelated, I think, in the end).

@rubensworks
Copy link
Member

Probably depends on #135.

@surilindur Can you rebase this one once #135 is merged?

@surilindur surilindur force-pushed the chore/dependencies-and-cleanup branch from dcd1781 to b734be6 Compare March 20, 2025 15:03
@surilindur
Copy link
Contributor Author

@rubensworks This should now be ready for review!

@rubensworks
Copy link
Member

Thx!

@rubensworks rubensworks merged commit be1cea7 into LinkedSoftwareDependencies:master Mar 20, 2025
21 checks passed
@surilindur surilindur deleted the chore/dependencies-and-cleanup branch March 20, 2025 15:45
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

Successfully merging this pull request may close these issues.

2 participants