Skip to content

Adds hack/dev-branch.sh - #775

Open
Bowei Du (bowei) wants to merge 1 commit into
agent-substrate:mainfrom
bowei:dev-branch
Open

Adds hack/dev-branch.sh#775
Bowei Du (bowei) wants to merge 1 commit into
agent-substrate:mainfrom
bowei:dev-branch

Conversation

@bowei

Copy link
Copy Markdown
Collaborator

This script automates creating an isolated dev branch in a git worktree for development. It also creates a separate Kind cluster for e2e development for the environment.

Adds agent skill to use the isolated dev branch for work.

This script automates creating an isolated dev branch in a
git worktree for development. It also creates a separate Kind
cluster for e2e development for the environment.

Adds agent skill to use the isolated dev branch for work.
This skill provides workflow guidance for executing concurrent development tasks,
working on separate features within the `agent-substrate/substrate` repository.

By isolating each feature inside its own **Git worktree** and **Kind Kubernetes

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Suggested change
By isolating each feature inside its own **Git worktree** and **Kind Kubernetes
By isolating each feature inside its own **Git worktree** and **kind Kubernetes

if [ "${reg_created_by}" == "agent-substrate" ]; then
echo "Deleting registry container '${reg_name}' (created by us)..."
docker rm -f "${reg_name}" || true
remaining_clusters="$("${ROOT}"/hack/kind.sh get clusters 2>/dev/null | grep -v 'No kind clusters found' | grep -v '^$' || true)"

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

this script was intended to be the counter to create-kind-cluster.sh, I think if we're going to delete all clusters that should be optional or a mode

currently if you are doing other things on the same host with kind, we won't delete that, just the one you requested. now this PR wipes unrelated clusters

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants