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

itzmidinesh
Hi all, I’m working on a Phoenix project where I need to make external HTTP requests and I’m trying to choose between Req and HTTPoison....
New
yarrichar
I’m very new to Elixir, so sorry if this is a silly question: I am wondering what happens to the mailbox of a process when the machine i...
New
pragdave
How do I tell Hologram not to escape HTML when substituting into templates? {Textile.render(@product.long_description)} escapes the HTM...
New
Fl4m3Ph03n1x
Background When trying to execute mix release on a Windows 11 machine for a Phoenix project I get the following error: * assembling mark...
New
rlipscombe
I’d like to create a file with 600 permissions (rw-------), but there’s a race condition if I create it and then change the permissions. ...
New
saleyn
Is there a way to import record definitions in the iex shell for pretty printing (similar to the rr command in the erl shell)? This woul...
New
rlipscombe
When I use Mix.install with a git dependency, using git@github.com:... how do I change the SSH identity used? I’ve got two Github accoun...
New

Other Trending Topics Top

dominicletz
Hey there, I haven’t seen a list of useful Elixir specific skills for the popular AI tools yet so I’m starting one here with my first. ...
New
raa-ziq
Hi everyone :waving_hand: I wanted to share something I’ve been building — NowBlind, a random chat app designed to help people discover ...
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
OndrejValenta
(I just needed to vent somewhere and LinkedIn is full of hope, or hype, I’m not sure which exactly) AI dream has many faces, but general...
New
kip
Localize is the next generation localisation library for Elixir. Think of it as ex_cldr version 3.0. The first version will be released ...
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

We're in Beta

About us Mission Statement

Options

Thread Display Mode




Thread Preview

Skip Thread Previews