revoltchat
BREAKING: use Deno tooling
This pull request changes a lot about revolt.js, mostly in terms of dependencies and how to build it. Most of the changes here are done to both remove dependencies and support Deno more.
This includes changes from #112, #113, and #114 as well as the following:
- BREAKING: use Deno tooling
- chore: add support for both npm and JSR publishing
In addition to these changes, https://github.com/insertish/oapi/pull/3 removes the axios and lodash.defaultsdeep dependencies from oapi and revolt-api by proxy.
That combined with this PR should make for both Node, Deno, and browser support while publishing to both npm and JSR with just three dependencies:
-
@vladfrangu/async_event_emitter -
revolt-api -
ulid
This fixes #34
Please make sure to check the following tasks before opening and submitting a PR
- [X] I understand and have followed the contribution guide
- [X] I have tested my changes locally and they are working as intended
I think it'd be a decent idea to split this up if possible? some of these changes look smaller/easier to merge
yeah, i'm going to work on splitting this up into a few other PRs. i'll keep this one open until i'm done with that
i've split some of the changes from this into https://github.com/revoltchat/revolt.js/pull/112, https://github.com/revoltchat/revolt.js/pull/113, and https://github.com/revoltchat/revolt.js/pull/114 but I'm still keeping this pr for specifically the Deno tooling since git has been driving me insane
since two of the split up prs have been merged, this and #114 (which this should be based on) have quite a lot of merge conflicts but i'll try and resolve those on my end
given so much has changed since I first made this pull request, and since some of the main goals of my work have already been merged, I'm going to close this. if anyone has remaining interest in this stuff getting upstream, I'd be happy to take another stab at it in another PR.