Fix wrong variable name used in argument parsing by wpamitkumar · Pull Request #5564 · wp-cli/wp-cli
Navigation Menu
{{ message }}
- Notifications You must be signed in to change notification settings
- Fork 1k
Merged
schlessera merged 1 commit intowp-cli:masterfrom
Nov 11, 2021Merged
Fix wrong variable name used in argument parsing#5564
schlessera merged 1 commit intowp-cli:masterfrom
Fix wrong variable name used in argument parsing#5564
schlessera merged 1 commit intowp-cli:masterfrom
Conversation
Copy link
Contributor
wpamitkumar
commented
Oct 1, 2021
wpamitkumar
commented
As per issue #5558 Fixed unused variable $positional_arg in foreach loop and replaced to $positional variable.
wpamitkumar
requested a review
from a team
as a code owner
swissspidy approved these changes Oct 1, 2021
schlessera approved these changes Nov 11, 2021
schlessera
added
bug
scope:framework
labels
schlessera
added this to the
2.5.1 milestone
schlessera
merged commit
f2294a4
into
wp-cli:master
Copy link
Member
schlessera
commented
Nov 11, 2021
schlessera commented
Nov 11, 2021Thanks for the PR, @wpamitkumar !
schlessera
mentioned this pull request
schlessera
changed the title
Fixed wrong variable used in parsing args.
Fix wrong variable name used in argument parsing
This file contains hidden or 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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment