Skip to content

feat!: remove dummy dataset search (testing reconcile pipeline)#7

Merged
ciaransweet merged 2 commits into
mainfrom
remove-hardcoded-dataset-search
Jun 16, 2026
Merged

feat!: remove dummy dataset search (testing reconcile pipeline)#7
ciaransweet merged 2 commits into
mainfrom
remove-hardcoded-dataset-search

Conversation

@ciaransweet

Copy link
Copy Markdown
Contributor

Testing the reconcile login by running scripts/remove-toolset dataset-search

@ciaransweet
ciaransweet force-pushed the remove-hardcoded-dataset-search branch from cbc4d08 to c85b23f Compare June 12, 2026 13:14

@yellowcap yellowcap left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

lgtm

Comment thread scripts/remove-toolset
echo "Not removed automatically: out-of-band Secrets the toolset listed in"
echo "toolset.yaml, and its images in GHCR."

# Other code or docs may still reference the toolset (README examples, tests

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Not sure if everyone will remember running this when removing a tool. But its ok to have it.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Yeah, the block below searches for references that weren't deleted and tells the invoker that 🫡

@ciaransweet
ciaransweet merged commit e966aef into main Jun 16, 2026
6 checks passed
@ciaransweet
ciaransweet deleted the remove-hardcoded-dataset-search branch June 16, 2026 14:38
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.

2 participants