-
Notifications
You must be signed in to change notification settings - Fork 0
Use submodules to maintain a long-term fork of Overtype #31
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
…platform (dev on apple silicon, CI on linux x86_64)
- build:dev for `wxt build --mode development`
@WebsByTodd this is ready! If you check it out and run
I'm not merging this PR because things are overall worse, specifically Once you get a chance to bring this PR up to parity with main then we can merge this. |
Oh! Also, I tried to use "npm workspaces", I did not want to switch to pnpm. But workspaces has bugs / missing features related to platform-specific stuff. So developing on Mac Silicon and running CI on linux was bad. I fixed/worked-around one issue, and then hit another one. That was when I switched to pnpm, which just worked. So that's how we got stuck with pnpm. |
Overtype is designed for creating new text areas, it is currently pretty awkward for modifying existing text areas. Hopefully we can eventually prepare a succinct PR that the upstream project will be willing to merge, but in the meantime we will need to maintain a fork for a while.
packages/overtype
and is hosted at https://github.com/diffplug/overtypemain
, plus we ignoreddist
, plus we merged feat: support codeblock syntax highlighting panphora/overtype#35