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

Export alt values in hooks and bootstrap #312

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

erijo
Copy link
Contributor

@erijo erijo commented Jan 17, 2021

What does this PR do?

Rename the local_* variables to YADM_* and export them so that hooks and bootstrap can use them.

What issues does this PR fix or reference?

Previous Behavior

Hooks and bootstrap had to come up with other ways to get the info.

New Behavior

Now they can just use YADM_* from the environment.

Have tests been written for this change?

Yes (a basic one)

Have these commits been signed with GnuPG?

Yes


Please review yadm's Contributing Guide for best practices.

kgadek added a commit to kgadek/yadm-bin that referenced this pull request Feb 21, 2021
kgadek added a commit to kgadek/yadm-bin that referenced this pull request Aug 8, 2021
kgadek added a commit to kgadek/yadm-bin that referenced this pull request Dec 2, 2021
local_class -> YADM_CLASS
local_system -> YADM_OS
local_host -> YADM_HOSTNAME
local_user -> YADM_USER
local_distro -> YADM_DISTRO

The new names match the names used for j2/esh templates.
kgadek added a commit to kgadek/yadm-bin that referenced this pull request Apr 17, 2022
kgadek added a commit to kgadek/yadm-bin that referenced this pull request Apr 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants