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

[Toolchain] Support multiple deno versions and os/arch #1

Open
TheGrizzlyDev opened this issue Jun 12, 2021 · 13 comments
Open

[Toolchain] Support multiple deno versions and os/arch #1

TheGrizzlyDev opened this issue Jun 12, 2021 · 13 comments

Comments

@TheGrizzlyDev
Copy link
Owner

TheGrizzlyDev commented Jun 12, 2021

Support multiple deno versions through a configurable build setting.
Support other version of os/arch other than linux/x86_64.

Nice to have:
toolchain download setup should be auto generated
list of versions should be limited to the ones actually supported

@alexeagle
Copy link

I have spent some time on this https://github.com/aspect-build/rules_deno/
Maybe we could collaborate on it?

@TheGrizzlyDev
Copy link
Owner Author

That would be lovely and I've been wanting to pick up this project again for a while. Would you be alright if we created an org and merged the 2 projects together? I was thinking of other tools for this ecosystem (like an equivalent to gazelle) in the future and this would keep things clean.

@alexeagle
Copy link

Of course I'm fond of keeping my rules repos in aspect-build org, but I can see why Engflow wouldn't want to sign a CLA there and I'm not tied to it.
github.com/bazel-contrib would be the next obvious GH org to be the home for it

Also I should be clear, I have very little time to contribute to this, and I'm also not sure if there's enough potential adoption to make it worth spending time.

@TheGrizzlyDev
Copy link
Owner Author

Sure, bazel-contrib would be a perfect place for it!

Same for me, I plan on contributing to this mostly on my spare time, so not much. How would you like to proceed then?

@alexeagle
Copy link

technically we need bazel-contrib/SIG-rules-authors#3 first

@TheGrizzlyDev
Copy link
Owner Author

  1. Must have a clear owner (POC) - Either me or you would be fine, I don't mind
  2. Must have clear readme outlining the goal of these rules, how to use them etc. - both repos have something in place, but I think we can improve a bit
  3. Must have examples - we both have some, my repo has a few more granular tests that we can merge in
  4. Must have tests that are running continuously - done
  5. Must have an SLA of 2-3 weeks (exact timing TBD) - let's have a chat about this, but 2-3 weeks would be ok for me
  6. Maybe: must have at least one more person that can approve PRs? - done
  7. Maybe: must follow the same release pattern? - we can adopt it
    Anything else I could be missing?

@beingminimal
Copy link

Is there any update on this?

@TheGrizzlyDev
Copy link
Owner Author

Is there any update on this?

I'm about to start some work so that we can match the criteria for bazel-contrib and soon after maybe @alexeagle can help merging their rules in? Regardless I will soon start getting some work done on this @codecycleteam

Just out of curiosity, do you plan on using these rules for some project? If that was the case it'd be interesting to get some feedback and conversations going :)

@beingminimal
Copy link

@TheGrizzlyDev Thank you for the updates!
Yes, we are planning to use deno with bazel in our internal project. And definitely we would like to give feedback.
Mainly we are looking forward for the bazel rules for deno bundle and test command with latest version of deno toolchain.

@alexeagle
Copy link

alexeagle commented Apr 22, 2022

@TheGrizzlyDev WDYT about starting from the commits on the aspect-build repo? It started from an earlier version of bazel-contrib/rules-template - I'll update it now to include the latest.

edit: take a look at https://github.com/aspect-build/rules_deno/pull/4/files ?

@TheGrizzlyDev
Copy link
Owner Author

TheGrizzlyDev commented Apr 22, 2022

@alexeagle sounds good to me, I don't mind which repo we use while migrating over to bazel-contrib. What features from mine would you need me to port over?

The PR lgtm, nice work!

@alexeagle
Copy link

Okay, merged that.
Filed the "process ticket" to get approval from the SIG to transfer the repo to bazel-contrib: bazel-contrib/SIG-rules-authors#48

As for what to add, looks like it's all your starlark rules, the deno_library, deno_binary, deno_test. Want to start with a PR against aspect-build/rules_deno?

@TheGrizzlyDev
Copy link
Owner Author

sgtm, thanks a lot for your help!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants