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

stjefim
Hello! Suppose you are building workflow (order / task / payment) processing system with the following requirements: Each workflow con...
New
water
I’m aware of 2 such projects, are there others? git: GitHub - team-alembic/realworld: A fullstack Phoenix LiveView application with back...
New
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
peck
Hi everyone. My team and I have been working on a fairly modest app based around video streaming and chat, but we’ve landed a customer t...
New
bradley
I really like the adapter patterns that ecto, nebulex, waffle, etc. use and would love find something similar for a key management servic...
New
unaware8150
Hello folks! So at work, we are seeing some situations where we have to define some “fixed” strings that are used across the codebase in...
New
Alvinkariuki
How Can I Optimise Compile Time Dependencies I have been building an elixir application for about 2 years now. Many modules and files ha...
New

Other Trending Topics Top

KristerV
Hey. Is there anyone here who creates agents in their apps? Not talking about using agents, but creating them. I’m finding it pretty diff...
New
mudasobwa
I fully migrated to my own harness from Anthropic/Gemini and I think it’s time to share it. Welcome DSH, the DeepSeek Harness, fully writ...
New
jimsynz
Beam Bots (or just BB for short) is a framework for building fault-tolerant robotics applications in Elixir using familiar OTP patterns. ...
New
juhalehtonen
There has been a thread to discuss the Stack Overflow Developer Survey on this forum every year since 2018, so here’s yet another one for...
New
budgie
I love Elixir. It’s one of 2 programming languages I’ve ever fallen in love with. But I don’t use it anymore. Serverless was the promis...
New
ausimian
Emily is an Elixir library that runs Nx computations on Apple’s MLX. Install it as the default Nx backend and Nx, defn, Axon, Nx.Serving,...
New

We're in Beta

About us Mission Statement

Options

Thread Display Mode




Thread Preview

Skip Thread Previews