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

feat: add tests workflow for bun and deno #423

Open
wants to merge 40 commits into
base: feature/deno
Choose a base branch
from

Conversation

nazarhussain
Copy link
Contributor

@nazarhussain nazarhussain commented Nov 21, 2024

Motivation

Make the packages compatible with Deno+Bun.

Description

  • Make unit tests compatible with Deno+Bun
  • Make spec tests compatible with Deno+Bun

Steps to test or reproduce

  • Run all tests

@nazarhussain nazarhussain changed the title Add tests workflow for deno chore: add tests workflow for deno Nov 21, 2024
@nazarhussain nazarhussain changed the title chore: add tests workflow for deno feat: add tests workflow for deno Nov 21, 2024
@nazarhussain nazarhussain marked this pull request as ready for review November 21, 2024 13:54
@nazarhussain nazarhussain requested a review from a team as a code owner November 21, 2024 13:54
@nazarhussain nazarhussain self-assigned this Nov 22, 2024
@twoeths
Copy link
Contributor

twoeths commented Nov 25, 2024

while I see the improvement on the source code, like not to import lib from other modules, I don't see why do we have to pick Deno and make sure lodestar to be compliant to it? there are a lot of deno specific resources added in this PR

I guess we need to put this on hold until we have final approach on which javascript runtime that lodestar want to experiment ChainSafe/lodestar#7237

@nazarhussain
Copy link
Contributor Author

nazarhussain commented Nov 25, 2024

while I see the improvement on the source code, like not to import lib from other modules, I don't see why do we have to pick Deno and make sure lodestar to be compliant to it? there are a lot of deno specific resources added in this PR

Deno is needed/recommended to publish packages to jsr.io which is intention to test with deno first in this PR.

And here is the details that why I am moving to support jsr.io https://jsr.io/docs/why

@twoeths
Copy link
Contributor

twoeths commented Nov 26, 2024

Does Bun support jsr.io?

if yes I'd go with Bun as we'll most likely to go with it. Again, need to wait for the final decision on ChainSafe/lodestar#7237

if both NodeJS + Bun don't support jsr.io, only Deno does then we need to see if it's a must to support jsr.io. I found it weird to target Bun/NodeJS while having Deno specific resources in the repo

@nazarhussain
Copy link
Contributor Author

Does Bun support jsr.io?

jsr.io is a registry developed and maintained by Bun for typescript based packages. As it follow modern standards it's compatible with all runtimes. As it's mentioned on there website.

JSR modules can be used in Node.js, Deno, Bun, Cloudflare Workers, and more.

@nazarhussain nazarhussain changed the title feat: add tests workflow for deno feat: add tests workflow for bun and deno Dec 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants