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

(RK-378) Restore access to the environment name from the Puppetfile #1241

Merged

Conversation

jarretlavallee
Copy link

This PR restores the basedir getter in the Puppetfile DSL with a deprecation warning and adds a new 'environment_name' getter that can be used to get the name of the environment.

Prior to this commit, accessing '@librarian.basedir' from the Puppetfile
was removed in a non major version. This commit restores the
'@librarian.basedir' functionality with a deprecation warning.
@jarretlavallee jarretlavallee requested a review from a team November 5, 2021 17:54
Prior to this commit, there was not an easy way to access the
environment name from the Puppetfile. This commit adds
'environment_name' to the Puppetfile DSL to allow users to access the
environment name if it is provided. The common way to access that is
'@librarian.environment_name' which will be a string containing the name
of the environment or `nil` if there was no environment provided.
@jarretlavallee jarretlavallee force-pushed the rk_378-restore_librarian_basedir branch from 3c684c4 to 7c9b582 Compare November 5, 2021 18:13
Copy link
Collaborator

@Magisus Magisus left a comment

Choose a reason for hiding this comment

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

Thanks for doing this, Jarret! I'll open a ticket for removing the old version and add it to our r10k 4 epic.

@Magisus Magisus merged commit ca519fb into puppetlabs:main Nov 5, 2021
@Magisus
Copy link
Collaborator

Magisus commented Nov 5, 2021

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