Skip to content

Move {init,create}GitRepo to tests/functional/common/functions.sh#14959

Merged
edolstra merged 1 commit intomasterfrom
git-repo-tests
Jan 9, 2026
Merged

Move {init,create}GitRepo to tests/functional/common/functions.sh#14959
edolstra merged 1 commit intomasterfrom
git-repo-tests

Conversation

@edolstra
Copy link
Member

@edolstra edolstra commented Jan 9, 2026

Motivation

This gets rid of some duplicate code in the tests.

Context


Add 👍 to pull requests you find important.

The Nix maintainer team uses a GitHub project board to schedule and track reviews.

@edolstra edolstra enabled auto-merge January 9, 2026 13:58
@github-actions github-actions bot added the with-tests Issues related to testing. PRs with tests have some priority label Jan 9, 2026
Copy link
Contributor

@xokdvium xokdvium left a comment

Choose a reason for hiding this comment

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

Very nice, thanks!

@edolstra edolstra added this pull request to the merge queue Jan 9, 2026
Merged via the queue into master with commit 477aa25 Jan 9, 2026
18 checks passed
@edolstra edolstra deleted the git-repo-tests branch January 9, 2026 14:50
JustAGuyTryingHisBest pushed a commit to JustAGuyTryingHisBest/nix that referenced this pull request Jan 12, 2026
Move {init,create}GitRepo to tests/functional/common/functions.sh
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

with-tests Issues related to testing. PRs with tests have some priority

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants