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

Fixup action name for the GH marketplace #287

Merged
merged 1 commit into from
Jun 16, 2024

Conversation

starbelly
Copy link
Member

@starbelly starbelly commented Jun 16, 2024

We seemed to have published to the marketplace previously with a description of the action vs the action name. This commit changes the name to simply setup-beam.

The link that is out there : https://github.com/marketplace/actions/setup-erlang-otp-with-optional-elixir-and-mix-and-or-rebar3

Additionally, if you search for setup-beam or beam this will action not come up. We may want to revise the description while we are here.

We seemed to have published to the marketplace previously with
a description of the action vs the action name. This commit changes the name
to simply `setup-beam`.
@paulo-ferraz-oliveira
Copy link
Collaborator

paulo-ferraz-oliveira commented Jun 16, 2024

I did the first change to the name, and at that time I didn't look at the action.yml reference. The description was previously "Setup Elixir" 😄, here.

@paulo-ferraz-oliveira
Copy link
Collaborator

Something like

  This action's goal is to set up a BEAM-based CI environment, with
  Erlang/OTP as a base, then optionally, Elixir, Gleam, `rebar3` or `hex`
  on top of that

?

I'm also Ok merging this now and changing the description in a subsequent pull request.

@paulo-ferraz-oliveira
Copy link
Collaborator

paulo-ferraz-oliveira commented Jun 16, 2024

Bryan, I'm merging this to check if the verification process moves forward, and then I'll pr to fix the description 😄 I'm eager to see if this unblocks the verification process 😄

@paulo-ferraz-oliveira paulo-ferraz-oliveira merged commit b7b9490 into erlef:main Jun 16, 2024
65 checks passed
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

Successfully merging this pull request may close these issues.

2 participants