Skip to content

update solf config bat#1121

Merged
milanmajchrak merged 1 commit intocustomer/vsb-tuofrom
update-solr-config-bat
Nov 4, 2025
Merged

update solf config bat#1121
milanmajchrak merged 1 commit intocustomer/vsb-tuofrom
update-solr-config-bat

Conversation

@Paurikova2
Copy link
Collaborator

Phases MP MM MB MR JM Total
ETA 0 0 0 0 0 0
Developing 0 0 0 0 0 0
Review 0 0 0 0 0 0
Total - - - - - 0
ETA est. 0
ETA cust. - - - - - 0

@Paurikova2 Paurikova2 linked an issue Oct 15, 2025 that may be closed by this pull request
@Paurikova2 Paurikova2 requested a review from Copilot October 15, 2025 07:07
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR updates a Windows batch script to properly remove Solr configsets before copying new ones. The change fixes command syntax by replacing Unix-style rm -rf commands with Windows-compatible rd /s /q commands and adds a descriptive comment.

  • Replaced Unix rm -rf command with Windows rd /s /q commands
  • Added explicit removal of each configset directory separately
  • Added a comment explaining the copy operation

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@milanmajchrak
Copy link
Collaborator

@Paurikova2 This should be also propagated into dtq-dev, right?

@Paurikova2
Copy link
Collaborator Author

@milanmajchrak yes

@Paurikova2
Copy link
Collaborator Author

@milanmajchrak
tested, you can merge

@milanmajchrak milanmajchrak merged commit 5932e53 into customer/vsb-tuo Nov 4, 2025
19 of 26 checks passed
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.

VSB-TUO/update-solr-script-bat

3 participants