Thanks to visit codestin.com
Credit goes to github.com

Skip to content

gh-91629 Use conf.d configs and fish_add_path to set the PATH when installing for the Fish shell. #91630

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

Merged
merged 5 commits into from
Apr 21, 2024

Conversation

drewcassidy
Copy link
Contributor

@drewcassidy drewcassidy commented Apr 17, 2022

addresses #91629 by writing path-setting configs to conf.d and using fish_add_path

• Use conf.d instead of modifying config.fish
• Use fish-add-path instead of modifying $PATH directly to avoid duplicates
@drewcassidy drewcassidy requested a review from a team as a code owner April 17, 2022 08:10
@ghost
Copy link

ghost commented Apr 17, 2022

All commit authors signed the Contributor License Agreement.
CLA signed

@erlend-aasland
Copy link
Contributor

@ned-deily, leaving this to you :)

@iritkatriel iritkatriel requested a review from ned-deily November 5, 2022 14:20
@drewcassidy
Copy link
Contributor Author

Hey uh, this PR is nearing its second birthday, any chance of it getting reviewed?

Copy link
Contributor

@erlend-aasland erlend-aasland left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Still looks good to me. I'll land this next week, to give Ned a chance to review it :) cc. @ned-deily

@erlend-aasland erlend-aasland self-assigned this Apr 15, 2024
@ned-deily ned-deily merged commit 5fa5b7f into python:main Apr 21, 2024
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.

4 participants