-
Notifications
You must be signed in to change notification settings - Fork 0
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
Discuss initial details around the action #1
Comments
I asked. "
" |
Robert replied. "
We can probably start with the ELP version only. Eqwalizer is packaged in there. The version has the format
Binaries are published as release assets. Example here. We currently provide binaries for Linux ( We probably need to depend on All the magic happens here.
I would expect the |
I've moved, to this repo, what I'd done in https://github.com/paulo-ferraz-oliveira/gha-example. Next I'll adapt the content to not mention "gha-example", and "Paulo", and then I'll try a simple Linux-working version of the installer with options. As I understand it now there's no option to guess the version (so we need it to be complete), but I can think of something later - mind you that most consumers don't care to update versions constantly which is why Also, for the macOS do you mean the other way around (ARM not AMD?). Just checking since most recent versions of macOS only run on ARM (right?). |
I've send you both invitations to collaborate, but don't know if I'll have to change them to "write" later. |
@paulo-ferraz-oliveira Giving you access to the private repo I created... |
I'll move it later, if that's Ok, since I wanna try out stuff with some Settings, too, and I might not be able to do so so easily next to |
@robertoaloi, is the name for the assets fixed? Example:
The version compatibility matrices can also be discussed later (#4), but I'm mostly interested in the Also, does it make a difference if it's Ubuntu 20, 22 or 24, as per GitHub's current runners? |
Note: Edit: also searched https://github.com/WhatsApp/eqwalizer/blob/main/FAQ.md, and https://github.com/WhatsApp/eqwalizer/tree/main/docs. |
I found a potential improvement on the CI file you linked (https://github.com/WhatsApp/erlang-language-platform/blob/main/.github/workflows/ci.yml#L87). |
You could also just set the remote for origin and do a |
Nah, because I have issues now, and I don't wanna deal with moving that too. 😄 Right now lemme focus on getting a first draft out and then we'll see. I just merged to the main branch. I'd be Ok for you to move the project now (or lemme know how/if I can do it). I don't wanna loose the issues (or have to copy them by hand) or the pull requests and the associated history. |
@robertoaloi, regarding moving forward: the action has I see:
I'm not sure how easy it'd be to pack all of this in a container (it'd also make consuming in non-GHA easier (?)). |
Yes. I was referring to this issue which is being fixed as we speak: WhatsApp/erlang-language-platform#34 |
The target names comes from the parameter passed to the
To get the list of possible targets.
|
@VLanvin is the eqWAlizer expert. I think most of eqwalizer is already pre-packaged in ELP so maybe installing the GraalVM is enough? Not sure which other installation steps would be required. Did you try to run |
That means you are running it on a project without a Documentation here. Note that I already pushed the build_info plugin to rebar3 itself, but we still need to modify ELP to use the native version. |
The |
You can just download the You however need to make |
Yeah, but it failed with some reason I can't remember. It's possible to create a pull request in this repo, change I'll also try to look at the TOML file you mention. Good stuff on pushing to
I'm Ok keeping it, if it's a rule, but I don't think it adds much in terms of semantics. |
Ref.: erlef/setup-beam#283
@robertoaloi, @starbelly, let's continue here. I'll try to keep issues to a minimum to move fast.
The text was updated successfully, but these errors were encountered: