-
Notifications
You must be signed in to change notification settings - Fork 39
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
actually do some maintenance #1697
Comments
Also, update networking code? |
Standard auth: https://webauthn.guide |
Something that might help with long-term maintenance,
|
Use MessageChannels instead of worker-bi ¹ https://developer.mozilla.org/en-US/docs/Web/API/AbortController from @sukima: function asyncPostMessage<t = unknown>(worker: Worker, message: unknown, transfers: Transferable[] = []): Promise<T> {
let { port1, port2 } = new MessageChannel();
return new Promise<T>((resolve) => {
port1.onmessage = ({ data }: MessageEvent<T>) => resolve(data);
worker.postMessage(message, [port2, ...transfers]);
});
} and in the worker: self.addEventListener('message', ({ data, ports: [port] }: MessageEvent<…>) => {
// Do stuff with data
port.postMessage(result);
}); |
As far as gjs/gts and v2 addons are concerned, a lot can be done to improve this repo' including adding turborepo to speed up ci.
I should do some of this and modernize this codebase.
Thankfully, it's not tooooo big.
The text was updated successfully, but these errors were encountered: