DEV Community

Devansh Shah
Devansh Shah

Posted on

Fork Issue Pull Merge

Path Normalized

The first repo I forked was while I went over the code I found that the file reader was only taking 1 type of file pat. So, I thought of normalizing the path and to do this I added the path dependency and used the method path.normalize(); with this normalizing the path. I also added the path dependency into the package.json. You can see the accepted pull request here .

Optimized Head

The Second repo I forked was After, I read through the repo I saw that when fetching the status code of links. This person was using the default get request. I found they were only using the status code so I thought a head request might be more efficient. So, I simply added a object param {method:'HEAD'} which should increase the speed at the links are fetched. Here is the pull request

Discussion (0)