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

Fix hard-coded username in pipenv installation instructions #335

Merged
merged 1 commit into from
Mar 20, 2024

Conversation

ppb2020
Copy link
Collaborator

@ppb2020 ppb2020 commented Mar 14, 2024

Copy/pasting the instructions for installing pipenv fail because of a hard-coded /home/user, forcing user to modify the instructions before executing the command. This can be addressed by using the ${USER} environment variable.

Signed-off-by: Pierre Pierre Blais ppblais@blackberry.com

Fix hard-coded username

Copy/pasting the instructions for installing pipenv fail because of a hard-coded /home/user, forcing user to modify the instructions before executing the command. This can be addressed by using the ${USER} environment variable.

Signed-off-by: Pierre Pierre Blais <ppblais@blackberry.com>
@erikbosch
Copy link
Collaborator

The failed DCO check likely depends on that the "author field" of the commit does not match sign-off. Have there possibly been any rebase/edit in Github UI. I anyway do not see this as a real blocker, so no reason to spend too much time on fixing it.

image

@ppb2020
Copy link
Collaborator Author

ppb2020 commented Mar 14, 2024

Weird; I used the same sign-off as I had used in my last commit...

Looks like if "ppb2020 [66322482+ppb2020@users.noreply.github.com]" was expected... Perhaps I should use this in the future?

@erikbosch
Copy link
Collaborator

Weird; I used the same sign-off as I had used in my last commit...

Looks like if "ppb2020 [66322482+ppb2020@users.noreply.github.com]" was expected... Perhaps I should use this in the future?

Nah, I think it just expects them to match. But I have seen "authors" like that previously when someone has done anything in the UI, like explicit/implicit rebase/squash and similar. But no big deal as I see it - there is a signoff and the signoff shows your real name so I consider the reported error more like a false positive

@ppb2020
Copy link
Collaborator Author

ppb2020 commented Mar 14, 2024

Ah, yes. As this was a very simple patch and only affecting documentation, I used the GitHub Web GUI to make the change as opposed to making the change locally on my development host. That was certainly a difference from my previous submissions.

@erikbosch
Copy link
Collaborator

MoM: Merge

@erikbosch erikbosch merged commit 9f71cbb into COVESA:master Mar 20, 2024
4 of 5 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.

2 participants