Eli
Docusaurus's ESM support is a sham![^1] They're just using Jiti, so this PR `require`s... the dreaded `eslint-disable` comment: unjs/jiti#101 [^1]: Very much an exaggeration, full of hyperbole: Jiti is certainly...
> About the unicorn changes, it's definitely a change I don't want. I'm pretty happy with how we are handling unicorn rules right now, i want to keep hand-picking them...
Superseded by #331.
Oh, just realize that this'll break #150 again. Shouldn't be hard fix though.
Took a look, we'll just need to add `"noExternal": ["@sherifforg/constants"]` to all of the `tsup.config.json` files. Speaking of which, they all need the same config. Do we just want to...
I realized I had a question about this: do we want sourcemaps? It'd be even smaller to get rid of them, and I don't see how they'd help debug for...
> > Took a look, we'll just need to add `"noExternal": ["@sherifforg/constants"]` to all of the `tsup.config.json` files. Speaking of which, they all need the same config. Do we just...
> > As `@sherifforg/constants` isn't there, it would succeed to install, but fail at runtime > > I don't think this is correct. Adding a dep as devDependency makes TSUP...
Ok, I looked at the source, it seems like the CLI defaults to not bundling node_modules, but the API does, and the default in the presence of a config is...
> @lishaduck also this needs to be updated and conflicts resolved. I'll get to it next week 🤞[^1] [^1]: Praying that the school IT department didn't botch anything up and...