escapedcat
escapedcat
Could this be a first iteration? Next step could be i.e. leave "Logout" on top right all the time and move currency switch to settings?
We do support comma by now: https://commitlint.js.org/concepts/commit-conventions.html#multiple-scopes But the delimiter can not be configured. Not sure if this is still needed though.
> when we update @commitlint/[email protected] , it doesn't work well as before Maybe open a separate new issue for this first. Everything that worked should still work the same.
Sure. Wanna create a new issue for this and create a PR? > First of all, thanks for your work on such an awesome linter! Thank @marionebl for that ❤️
Uhm, so I thought the difference is this: - This issue wants to make the delimiter configurable i.e. instead of the currently [supported defaults](https://commitlint.js.org/concepts/commit-conventions.html#multiple-scopes) the user wants to use `%`...
> [...] since you're familiar with the project’s codebase I'm really not 😆 > What do you think about that? Got it, yes, that would cover both of the points...
```sh Run yarn build yarn run v1.22.22 error @commitlint/[email protected]: The engine "node" is incompatible with this module. Expected version ">=v20". Got "18.19.1" error Commands cannot run with an incompatible environment....
Also tagging @kaiehrhardt @jeohist , sorry! Happy for feedback.
> What features are we going to use which are only available in Node 20+? Propably none? We used to switch versions when they ran out of maintenance. Should we...
Fair enough 👍