Add CLI data integrity repair tool #1725
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
anvit
force-pushed
the
dev/data-integrity-tool
branch
from
1b50dd8 to
6f68973
Compare
anvit
force-pushed
the
dev/data-integrity-tool
branch
from
6f68973 to
e501539
Compare
anvit
force-pushed
the
dev/data-integrity-tool
branch
from
d2d7e1f to
2ff4682
Compare
|
I don't know what are the plans for documentation, but we should explain somewhere the meaning of the CSV and the issues that can be found there. |
Add a CLI tool for performing data integrity checks and repairing or deleting broken records.
anvit
force-pushed
the
dev/data-integrity-tool
branch
from
2ff4682 to
b8be3e9
Compare
- Fix indentation - Change calls to log to logSection - Move duplicated calls to report outside the switch - Remove csv filename extension check - Update help message, and clarify the .csv file extension
anvit
force-pushed
the
dev/data-integrity-tool
branch
from
b8be3e9 to
6588b07
Compare
|
@jraddaoui Just addressed the feedback! Not sure why PHP-CS fixer didn't catch indentation issues because it seems to be catching other things in the file just fine, but those should be fixed now |
jraddaoui
approved these changes
Mar 19, 2024
Remove the stringEndsWith function which was unused
melaniekung
approved these changes
Mar 19, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Add a CLI tool for performing data integrity checks and repairing or deleting broken records.
(Adapted from @jraddaoui's data integrity script into a CLI task)