Skip to content

Conversation

@TylerWitt
Copy link
Contributor

Adds linting and tests to CI.

Copilot AI review requested due to automatic review settings November 19, 2025 18:48
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR introduces a GitHub Actions CI workflow to automate linting and testing for an Elixir project. The workflow runs on pushes to main and pull requests targeting main, executing format checks, compilation with warnings-as-errors, and tests.

Key changes:

  • Adds a new CI workflow with matrix testing across multiple OTP and Elixir versions
  • Implements dependency and build caching to improve workflow performance
  • Includes code formatting verification and test execution steps

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Adds linting and tests to CI.
@TylerWitt TylerWitt requested a review from fatcatt316 November 19, 2025 19:07
@TylerWitt
Copy link
Contributor Author

@fatcatt316 This addresses many of the comments in #154 because CI forced me to format as well as fix the duplicated @doc definitions.

# Step: Conditionally bust the cache when job is re-run.
# Sometimes, we may have issues with incremental builds that are fixed by
# doing a full recompile. In order to not waste dev time on such trivial
# issues (while also reaping the time savings of incremental builds for
Copy link
Contributor

Choose a reason for hiding this comment

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

This comment is helpful as heck!

@TylerWitt TylerWitt requested a review from fatcatt316 December 1, 2025 17:22
Copy link
Contributor

@fatcatt316 fatcatt316 left a comment

Choose a reason for hiding this comment

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

Thanks for adding this 🫡

@TylerWitt TylerWitt merged commit 01923eb into master Dec 1, 2025
3 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.

3 participants