Fix the TODOs in .github/workflows/nodejs.yml

[cclauss]

https://github.com/TheAlgorithms/Javascript/blob/master/.github/workflows/nodejs.yml

[raklaptudirm]

# TODO: error: Line 1: Unexpected token >>

We need to add doctests to all files to fix that issue.

[cclauss]

Or only run doctest on the files that have doctests.

[raklaptudirm]

Yes but I could not find any exclusion mechanism in the docs (like /* doctest exclude */). We can selectively run test is the workflow, but that means it has to be updated while adding any new algorithm. We need a better alternative.

The second TODO has been fixed here #609 .

[cclauss]

npx doctest a/b.js c/d.js e/f/g.js h/*.js

[kkanekii]

Pull request please

[raklaptudirm]

@kkanekii If you want to help, you can create a Pull Request.

Can someone guide me on creating a pull request?

I have to change this, right?

name: Node CI
on: [push, pull_request]
jobs:
  build:
    runs-on: ubuntu-latest
    strategy:
      matrix:
        node-version: [14.x]
    steps:
    - uses: actions/checkout@v2
    - name: Use Node.js ${{ matrix.node-version }}
      uses: actions/setup-node@v2
      with:
        node-version: ${{ matrix.node-version }}
    - name: npm install, build, and test
      run: |
        npm install doctest standard --save-dev
-       npx doctest **/*.js || true  # TODO: Add all doctests
+      npx doctest a/b.js c/d.js e/f/g.js h/*.js
        npx standard
        npm ci
        npm run build --if-present
        npm test
      env:
        CI: true

[cclauss]

Looks like you at least need another space.

What happens if you leave the command as it is but remove the || true?

We would rather run all the tests, not just hard code in the tests the exist today.

[defaude]

This will get resolved once we got rid of doctest altogether. See #742