Skip to content

fix: Allow --reset or --init on reindex_studio as no-ops#38434

Merged
kdmccormick merged 1 commit intomasterfrom
kdmccormick/reindex-studio-flags
Apr 23, 2026
Merged

fix: Allow --reset or --init on reindex_studio as no-ops#38434
kdmccormick merged 1 commit intomasterfrom
kdmccormick/reindex-studio-flags

Conversation

@kdmccormick
Copy link
Copy Markdown
Member

Follows up on:

Related:

Changing these to warnings will smooth out the transition for developers and operators.

@kdmccormick kdmccormick requested review from bradenmacdonald and farhaanbukhsh and removed request for farhaanbukhsh April 23, 2026 19:23
@kdmccormick kdmccormick force-pushed the kdmccormick/reindex-studio-flags branch from ddf0cb1 to 6049f99 Compare April 23, 2026 19:23
@kdmccormick
Copy link
Copy Markdown
Member Author

I'm not sure if this is the right fix to make, but I'll put it on #38402 temporarily just to fix my sandbox.

Copy link
Copy Markdown
Contributor

@bradenmacdonald bradenmacdonald left a comment

Choose a reason for hiding this comment

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

Yeah, I think this makes sense too.

@kdmccormick kdmccormick force-pushed the kdmccormick/reindex-studio-flags branch from 6049f99 to 05e3e2e Compare April 23, 2026 20:01
@kdmccormick kdmccormick enabled auto-merge (squash) April 23, 2026 20:03
@kdmccormick kdmccormick merged commit e2cd3df into master Apr 23, 2026
44 of 45 checks passed
@kdmccormick kdmccormick deleted the kdmccormick/reindex-studio-flags branch April 23, 2026 20:35
@farhaanbukhsh
Copy link
Copy Markdown
Member

@kdmccormick @bradenmacdonald thank you for taking care of this. 😄

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