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.

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

Fixed an issue with the --user option. #142

Closed
wants to merge 3 commits into from
Closed

Fixed an issue with the --user option. #142

wants to merge 3 commits into from

Conversation

chymb
Copy link

@chymb chymb commented Oct 27, 2021

The argument was never being used and causing the help/usage to show.
Feel free to implement it in your own style/way. I figured it was easier to fix it than log it as a bug.

grahampugh and others added 3 commits October 24, 2021 18:27
The argument was never being used and causing the help/usage to show.
@grahampugh grahampugh changed the base branch from main to 24.1-rc October 27, 2021 09:03
grahampugh added a commit that referenced this pull request Oct 27, 2021
@grahampugh
Copy link
Owner

Thank you! I've made the fix in the v24.1 branch (the PR didn't function well because it included changes to the Bug Report for some weird reason, but I've manually made the change).

@grahampugh grahampugh closed this Oct 27, 2021
@chymb
Copy link
Author

chymb commented Oct 27, 2021

Thanks. Sorry about that, I think I somehow applied the PR to that branch rather than main. 😳

@grahampugh
Copy link
Owner

No, I moved it to the branch, because I generally work from a release candidate branch rather than merge things directly into main.

@grahampugh
Copy link
Owner

that said, I just merged the v24.1 branch into main but immediately found another problem..... So there will be another patch directly to main shortly :)

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.

2 participants