silverwind

Thanks for searching. I was almost sure that [`Set.add`](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Set/add) would support it as well, but alas, no.

> We should add a dash to `classlist`. I take it this is consistent with other rule names?

Uninstalling minikube (`brew remove minikube`) and "Reset All Data" in the Orbstack's options seems to have fixed this for me. Edit: No: after restarting a affected container, it breaks again...

Is there a way to avoid having vite log this warning? I personally don't care about source maps for YAML files but I do care about warnings in the build...

I don't think sourcemaps are useful for this plugin because it outputs JS objects, where there is no way to reference a line back to the source file because JS...

Of note is that the warning does not originate in vite itself, but in [rollup](https://github.com/rollup/rollup/blob/3cb7f1376f4bbd519d320491629111e1d26cfc80/src/utils/error.ts#L747). Edit: also see https://rollupjs.org/guide/en/#warning-sourcemap-is-likely-to-be-incorrect

https://github.com/Modyfi/vite-plugin-yaml/pull/9 will suppress the warning until it is figured out how to actually generate a sourcemap.

Technically they are equal. I guess the only benefit of the `var` variant is that it may be easier to understand for someone coming from languages that do not have...

Sounds good. I would change the return type to accept both sync and async: ````ts export type OnNavigateFn = (data: OnNavigateData) => Promise | boolean; ```` Implementation should be the...

This UI closely mirrors what GitHub has and the primary color should indicate the primary action for a reviewer. The location is so that the reviewer is forced to look...