Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add --exclude argument for verifying checksums #123

Merged
merged 4 commits into from
Aug 7, 2024
Merged

Add --exclude argument for verifying checksums #123

merged 4 commits into from
Aug 7, 2024

Conversation

iDschepe
Copy link
Contributor

No description provided.

@iDschepe iDschepe requested a review from a team as a code owner June 13, 2024 14:44
@iDschepe iDschepe changed the title Feat/checksum added exclude files parameter for core Feat/checksum added exclude parameter Jun 13, 2024
@iDschepe iDschepe force-pushed the feat/checksum-exclude-files branch from 4013dc4 to 5dbc550 Compare June 13, 2024 14:52
@iDschepe
Copy link
Contributor Author

At line 61 at checksum-command/src/Checksum_Base_Command.php there are two unused parameters. I did not touch these function, but also not sure, if we can simply remove $key and $iterator

@mrsdizzie mrsdizzie mentioned this pull request Jun 13, 2024
Closed
@swissspidy swissspidy changed the title Feat/checksum added exclude parameter Add --exclude argument for verifying checksums Aug 7, 2024
@swissspidy swissspidy added this to the 2.3.0 milestone Aug 7, 2024
@swissspidy swissspidy merged commit 3313a6b into wp-cli:main Aug 7, 2024
34 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@swissspidy swissspidy swissspidy approved these changes

Assignees
No one assigned
Labels
command:core-verify-checksums
Projects
None yet
Milestone
2.3.0
Development

Successfully merging this pull request may close these issues.
2 participants
@iDschepe @swissspidy