Fix an infinite loop if rollup config can't be found.

Review Request #12956 — Created April 14, 2023 and submitted — Latest diff uploaded

Information

Djblets
release-4.x

Reviewers

The code to try to find the rollup configuration file iterates up
the directory tree until either the config is found or we hit the root.
Unfortunately, this was comparing a Path object (for the current path)
against str (for the root), which always evaluates to False. The
result was that if there's no rollup config, we'd end up infinitely
looping comparing Path('/') to /. This change fixes that.

Tried to load an extension which defined an index.ts file in its
bundles but did not have a rollup.config.js file. Saw that we no
longer hit the infinite loop.

Commits

Files

    Loading...