feat: add support for jsconfig.json

[F3n67u]

close #127

[F3n67u]

@jonaskello please help review this pr. thanks

[F3n67u]

@jonaskello please help review this pr. thanks

@jonaskello do you have time to review this feature?

[aleclarson]

It would be more efficient to use readdirSync here, which is only one I/O call instead of two existsSync I/O calls.

[jonaskello]

@F3n67u I've asked @aleclarson to review this one

[F3n67u]

It would be more efficient to use readdirSync here, which is only one I/O call instead of two existsSync I/O calls.

@aleclarson thanks for your time. I already use readdirSync instead of existSync

[aleclarson]

@jonaskello Was there a good reason for using path.join instead of path.dirname here?

[F3n67u]

• path.join('/root/dir', '../') return /root/
• path.dirname('/root/dir') return /root(without the trailing directory separators)

the latter is normalized, easier to mock when we mock the fs.readdirSync, so I adjust path.join to path.dirname; though path.join(directory, '..') is fine either.

[F3n67u]

@aleclarson could help take a look at other 2 pr I made?

[F3n67u]

@aleclarson I see you approved this pr. do you have enough permission to merge this pr?

[jonaskello]

@F3n67u Can you also update the README and CHANGELOG (under unreleased entry)? After that we should be good to merge.

[F3n67u]

@F3n67u Can you also update the README and CHANGELOG (under unreleased entry)? After that we should be good to merge.

all done! thanks for your time. @jonaskello

[MichaelDeBoey]

@aleclarson @dividab @jonaskello Any idea when this will be released?

[jonaskello]

We need to get #206 merged before next release because #205 is merged but needs to be changed by #206.

[MichaelDeBoey]

@jonaskello Would it be possible to temporarily revert #205 and cut a release already please?

This would help the work I'm doing on Remix Stacks (see remix-run/indie-stack#63, remix-run/blues-stack#57 & remix-run/grunge-stack#50) a lot.
That way we could use tsconfig-paths to load jsconfig too instead of trying out a custom solution.

[jonaskello]

@MichaelDeBoey This is released now in 4.0.0.

[MichaelDeBoey]

@jonaskello Are there any breaking changes?
Looking at the CHANGELOG there aren't, but wanted to check to be sure.

[jonaskello]

The breaking changes are listed under the "changed" heading in the changelog.

[aleclarson]

This has a bug. We should be setting allowJs to true for jsconfig.json files.