Skip to content
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

Migrate to Deno #286

Merged
merged 16 commits into from
Jan 1, 2025
Merged

Migrate to Deno #286

merged 16 commits into from
Jan 1, 2025

Conversation

Wissididom
Copy link
Collaborator

@Wissididom Wissididom commented Nov 6, 2024

This PR migrates to using Deno instead of NodeJS

@Wissididom Wissididom added the enhancement New feature or request label Nov 6, 2024
@Wissididom Wissididom changed the base branch from master to develop November 6, 2024 15:20
@Wissididom
Copy link
Collaborator Author

Wissididom commented Nov 7, 2024

When I've tested it, it'll have to wait for https://github.com/hasundue/molt-action/issues/33

@Wissididom
Copy link
Collaborator Author

Might be helpful for Docker stuff: https://www.youtube.com/watch?v=VRryNeYm6yw

@Wissididom
Copy link
Collaborator Author

I'm gonna finish this PR without molt, because apparently molt will get discontinued and until there is a solution for automatic updates, I think I can just do them manually regularly (maybe monthly), since the only dependencies are @std/assert, npm:discord.js, npm:pg and npm:sequelize

@Wissididom Wissididom marked this pull request as ready for review January 1, 2025 17:01
Copy link
Collaborator Author

@Wissididom Wissididom left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. I tested using by running /leaderboard and /explore once, both returned what I expected.
Screenshot From 2025-01-01 18-53-16

@Wissididom Wissididom merged commit 7a22af5 into develop Jan 1, 2025
9 checks passed
@Wissididom Wissididom deleted the use-deno branch January 1, 2025 19:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant