Skip to content

PR: Replace Travis CI with GitHub Actions issue #230#234

Merged
asntc merged 8 commits intomainfrom
replace-travis-with-github-acctions-issue#230
Apr 16, 2026
Merged

PR: Replace Travis CI with GitHub Actions issue #230#234
asntc merged 8 commits intomainfrom
replace-travis-with-github-acctions-issue#230

Conversation

@nelsonic
Copy link
Copy Markdown
Member

@nelsonic nelsonic commented Apr 15, 2026

Adding GitHub Actions to this repo has been an excellent opportunity to significantly reduce the amount of noise in the /examples and cut out most of the advanced examples that 99.9% of people will never read.
So I've taken this opportunity to delete the code. All of that sample code comes from http://www.pragmaticprogrammer.com/titles/elixir13 (404) at the time of writing ... 🙄
The newer version is: https://pragprog.com/titles/elixir16/programming-elixir-1-6/
and a basic Google search will show you the PDF: google.com/search?q=Programming+Elixir+1.6+pdf and thus the code.
No need to duplicate it here.

Note: I've kept the more beginner-friendly examples for people to get a feel for the syntax. 🔰

Other than that, the PR is focussed on adding a test for a sample set of functions and adding Codecov.
Though I have not added badges at this time because the one that I care about is "broken": dwyl/hits#306

@nelsonic nelsonic added enhancement New feature or enhancement of existing functionality awaiting-review An issue or pull request that needs to be reviewed technical A technical issue that requires understanding of the code, infrastructure or dependencies elixir Pull requests that update Elixir code tech-debt A feature/requirement implemented in a sub-optimal way & must be re-written T5m Quick tasks that take 5 mins or less. See: GTD 2 min rule. labels Apr 15, 2026
@nelsonic nelsonic requested a review from asntc April 15, 2026 21:06
Copy link
Copy Markdown
Member

@asntc asntc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. :shipit:

@asntc asntc merged commit 684d790 into main Apr 16, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting-review An issue or pull request that needs to be reviewed elixir Pull requests that update Elixir code enhancement New feature or enhancement of existing functionality T5m Quick tasks that take 5 mins or less. See: GTD 2 min rule. tech-debt A feature/requirement implemented in a sub-optimal way & must be re-written technical A technical issue that requires understanding of the code, infrastructure or dependencies

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants