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

possible to specify where to deploy pyenvrc file #155

Merged
merged 2 commits into from
Jan 7, 2023
Merged

possible to specify where to deploy pyenvrc file #155

merged 2 commits into from
Jan 7, 2023

Conversation

MorganLindqvist
Copy link

Previously the pyenvrc file was placed in the pyenv root folder, this pollutes the git context.

Change enables placing the pyenvrc file in a different location, default is old location.

Copy link
Owner

@staticdev staticdev left a comment

Choose a reason for hiding this comment

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

@MorganLindqvist thanks a lot for your contribution and sorry for taking long to respond. Actually I did some work in parallel that conflicted. I would suggest you if possible to rebase your branch and rename the new parameter to pyenvrc_path this will be clear for the users since it exact name.

…r than pyenv_path

Change-Id: I79d7aec3be083ead6e09630da6ebaa7c35fd0311
Change-Id: Ib22ba6e03c23cb546af562b8d101fbaee5e41874
@MorganLindqvist
Copy link
Author

Rebased and parameter name changed as requested

@staticdev
Copy link
Owner

@MorganLindqvist LGTM.

@staticdev staticdev merged commit a89af57 into staticdev:main Jan 7, 2023
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