frigidcode

frigidcode

Does anyone have a Github action that will run Credo (or any command) against only the changed files for the branch/pull request? If so, I would greatly appreciate it :slight_smile: Thank you!

Showing Posts 1 to 6

tomkonidas

tomkonidas

Yes is would be something like this: mix credo diff --from-git-merge-base main where main is the branch you want to check for changes against.

This requires a version of credo >= 1.6. Here are some docs from the changelog were it was introduced.

13
Post #1
frigidcode

frigidcode OP

TIL - Thank you!!

frigidcode

frigidcode OP

I’m circling back to this and cannot get this to work within GH Actions. Using master / origin/master / ${{ github.head_ref }} all result with the error ** (diff) given value is not a Git ref: .

It has to do with how the code is checked out within a GH Action, I haven’t had enough time to dig into it, any help will be appreciated!

tomkonidas

tomkonidas

Can you provide us with your GH Action yaml?

frigidcode

frigidcode OP

on: push

jobs:
  formatting:
    runs-on: ubuntu-latest
    name: Format check
    steps:
      - uses: actions/checkout@v2
      - uses: erlef/setup-beam@v1
        with:
          otp-version: 23.3.4.4
          elixir-version: 1.12.1
      - run: mix deps.get
      - run: mix format --check-formatted
  credo:
    runs-on: ubuntu-latest
    name: Credo
    steps:
      - uses: actions/checkout@v2
      - uses: erlef/setup-beam@v1
        with:
          otp-version: 23.3.4.4
          elixir-version: 1.12.1
      - run: mix deps.get
      - run: mix credo diff --from-git-merge-base origin/master
  compilation:
    runs-on: ubuntu-latest
    name: Compilation with warnings
    steps:
      - uses: actions/checkout@v2
      - uses: erlef/setup-beam@v1
        with:
          otp-version: 23.3.4.4
          elixir-version: 1.12.1
      - run: mix deps.get
      - run: mix compile --warnings-as-errors

I plan on adding build and dep caching once I have it working, thanks!

frigidcode

frigidcode OP

I had some help from Twitter, and I needed to adjust the checkout action. It works now :slight_smile:

credo:
    runs-on: ubuntu-latest
    name: Credo
    steps:
      - uses: actions/checkout@v2
        with:
          fetch-depth: 0
      - uses: erlef/setup-beam@v1
        with:
          otp-version: 23.3.4.4
          elixir-version: 1.12.1
      - run: mix deps.get
      - run: mix credo diff --from-git-merge-base origin/master
— All posts loaded —

Where Next? Top

Trending in Questions Top

katta
I having some trouble figuring out if I have set myself too strict of standards for my production server. Currently I can handle 75% of r...
New
brecabral
Documentation While reading the Scoped Routes section, I noticed that the documentation currently refers to a problem without explainin...
New
achenet
Hello, I’m trying to build a basic Phoenix web-app, and I’d like to use Tailwind. However, when I launch mix phx.server, I get an error...
New
kpanic
Hi everyone, I am toying with the idea of building a “match maker” for giving personal help to people that wants to start coding. I sta...
New
asweet-confluent
I recently noticed that Elixir’s Logger defaults its primary log level to :debug when no :logger, :level application configuration is pre...
New
Cxx-mlr
I’m working on a small exercise involving update_in/3, and I came up with this solution: data = %{ name: "Periodic Table", category:...
New
ChrisAmelia
I’ve got trouble wrapping my head around the order in which functions are called in this snippet (from Phoenix’s authentication): toke...
New

Other Trending Topics Top

GenericJam
Edit: 2026 May 15 - This post is archived. Mob is alive!! Main docs: mob v0.7.11 — Documentation A bit of explanation for the slightly c...
New
JesseHerrick
Hey, I’m Jesse and I’m the main contributor behind Dexter, a full-featured, lightning-fast Elixir LSP optimized for large codebases. It s...
New
mudasobwa
I am happy to introduce the very α version of the new programming language compiled to BEAM. Welcome Cure. It has literally three kille...
New
garrison
Hobbes is a low-level distributed database for the Elixir programming language. Hobbes provides a simple, safe, and scalable storage lay...
New
mhanberg
Hi everyone! The first release candidate for the Expert language server project is now available! We’ve published a press release detai...
New
budgie
A little off-topic, but I feel like people here have a good head on their shoulders. I used to be quite good at making software. Was luc...
New

We're in Beta

About us Mission Statement

Options

Thread Display Mode




Thread Preview

Skip Thread Previews