2075800828
This patch proposes removing the `browserify` example for the following reasons: - The last `browserify` release was almost two years ago, according to both https://github.com/browserify/browserify/releases and https://www.npmjs.com/package/browserify?activeTab=versions - The project no longer seems to be actively maintained, since so far this year there's only been *a single* (seemingly trivial) patch merged; see https://github.com/browserify/browserify/commits/master - Because of the previous points `browserify` doesn't support modern and up-to-date JavaScript features, as evident from e.g. issue 14731 and multiple issues found in https://github.com/browserify/browserify/issues - Our `browserify` example is most likely not very commonly used, judging by the very low volume of issues/PRs related to it. Looking at the `git` history of that example the only changes have been lint- or maintenance-related.[1] - Providing an example for a framework that's no longer actively maintained doesn't seem like a good idea in general, since we probably don't want to steer users towards using (possibly) older frameworks. - Given that we've never used `browserify` in the PDF.js project, it's also quite difficult to provide support for the example. --- [1] It's interesting to compare with the `webpack` example, since that's generated both issues *and* also PRs (for missing features) from users. |
||
|---|---|---|
| .. | ||
| components | ||
| image_decoders | ||
| learning | ||
| mobile-viewer | ||
| node | ||
| text-only | ||
| webpack | ||
| .eslintrc | ||