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.

Sign up for GitHub

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

Jump to bottom

Github Actions and current Elixir support #73

Open
wants to merge 17 commits into
base: master
Choose a base branch
from
Open

Github Actions and current Elixir support #73

wants to merge 17 commits into from

Conversation

nlwstein
Copy link

@nlwstein nlwstein commented Feb 2, 2023 •
edited
Loading

Adds:

  1. Support for the current version of Elixir + Erlang/OTP (1.14/25)
  2. Moves CI to GitHub Actions, with addition of Hex package publishing (simply runs mix hex.publish --yes, requires secret HEX_API_KEY on repo).
  3. Bumps minimum version to 1.10

Examples:

  1. CI run: https://github.com/nlwstein/recaptcha/actions/runs/4071092141
  2. Coveralls: https://coveralls.io/github/nlwstein/recaptcha

Bits of code that meaningfully changed:

  1. Refactor requested by credo: e2c4a36
  2. Test modified due to what appears to be a Google API change: 7cc616c

nlwstein added 17 commits February 1, 2023 19:36
@nlwstein nlwstein marked this pull request as ready for review February 2, 2023 04:25
@nlwstein nlwstein mentioned this pull request Feb 2, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@nlwstein