Skip to content

ci: replace dasel with yq - #2367

Merged
burgerdev merged 2 commits into
mainfrom
burgerdev/dasel
May 6, 2026
Merged

ci: replace dasel with yq#2367
burgerdev merged 2 commits into
mainfrom
burgerdev/dasel

Conversation

@burgerdev

@burgerdev burgerdev commented May 4, 2026

Copy link
Copy Markdown
Member

@burgerdev
burgerdev requested a review from charludo May 4, 2026 07:47
@burgerdev burgerdev added the no changelog PRs not listed in the release notes label May 4, 2026
@burgerdev
burgerdev marked this pull request as draft May 4, 2026 09:14
@burgerdev
burgerdev marked this pull request as draft May 4, 2026 09:14
@charludo

charludo commented May 5, 2026

Copy link
Copy Markdown
Collaborator
  yq-go = prev.yq-go.overrideAttrs (
    finalAttrs: _prevAttrs: {
      version = "4.53.2";
      src = final.fetchFromGitHub {
        owner = "mikefarah";
        repo = "yq";
        rev = "v${finalAttrs.version}";
        hash = "sha256-So0yUblAxLgbw1/BrOp6zj9wGMTtsspjk2UQaBIBIYE=";
      };
      vendorHash = "sha256-t+u3rJNbe8JcfWBb+jMaZuTcSWYmhMiCeKbI2noVaCo=";
    }
  );

for the version. Sorry, not completely clear on what went wrong/what to test with the old version.

@burgerdev
burgerdev marked this pull request as ready for review May 5, 2026 12:41
@burgerdev
burgerdev merged commit fc0db13 into main May 6, 2026
84 of 95 checks passed
@burgerdev
burgerdev deleted the burgerdev/dasel branch May 6, 2026 08:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no changelog PRs not listed in the release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants