Jakub T. Jankiewicz

`fs` inside the library is not the same as `require('fs');`. It's a wrapper over `fs` library for internal use. If you want to contribute to a project, you first need...

This is the code for the `fs` used internally: https://github.com/isomorphic-git/isomorphic-git/blob/main/src/models/FileSystem.js

Whatever you implemented needs to be tested that it's correct. It can be one test or multiple tests if there are multiple use case that your feature addresses.

Also, can you rebase with the latest changes?

The file system is abstracted away. If one is working, it will work for the other. Submodules are related to the file system, so you don't need to test every...

But if you add your feature to 66 commands, then all should be tested.

Tests failed, and I don't see the longs anymore.

Don't worry about the errors, only isomorphic-git-PR is important, this is an old pipeline from Azure. I will need to review the code again.

I can't accept a PR where tests are disabled for browser.

The GitHub actions are not fully implemented and working. I think that we had problems with sharing secrets with forks. Also note that your branch don't start with `feature/`. If...