Skip to content

Conversation

iamriajul
Copy link

Description

Custom script to run after cloning the repository. Runs always after git clone, even if the repository already exists.

Type of Change

  • New module
  • Bug fix
  • Feature/enhancement
  • Documentation
  • Other

Module Information

Path: registry/coder/modules/git-clone
New version: v1.2.0
Breaking change: [ ] Yes [x] No

Testing & Validation

  • Tests pass (bun test)
  • Code formatted (bun run fmt)
  • Changes tested locally

Related Issues

@matifali matifali added the version:minor Add to PRs requiring a minor version upgrade label Aug 23, 2025
@DevelopmentCats
Copy link
Contributor

@iamriajul

These are from the logs of the failing test in git-clone

 Error: Expected: 128
  Received: 0
  
        at <anonymous> (/home/runner/work/registry/registry/registry/coder/modules/git-clone/main.test.ts:33:29)
  28 |     const state = await runTerraformApply(import.meta.dir, {
  29 |       agent_id: "foo",
  30 |       url: "fake-url",
  31 |     });
  32 |     const output = await executeScriptInContainer(state, "alpine/git");
  33 |     expect(output.exitCode).toBe(128);
                                   ^
  error: expect(received).toBe(expected)
  
  Expected: 128
  Received: 0
  
        at <anonymous> (/home/runner/work/registry/registry/registry/coder/modules/git-clone/main.test.ts:33:29)
  (fail) git-clone > runs with git [2524.00ms]

@DevelopmentCats
Copy link
Contributor

DevelopmentCats commented Sep 30, 2025

Once these are resolved I will test, and then we can merge

@DevelopmentCats
Copy link
Contributor

@iamriajul I just wanted to check in on this and see if there was an update?

@DevelopmentCats
Copy link
Contributor

@Parkreiner @matifali can one of you guys review this when you get a chance?

Copy link
Member

@Parkreiner Parkreiner left a comment

Choose a reason for hiding this comment

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

This looks good to me!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

version:minor Add to PRs requiring a minor version upgrade

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants