Skip to content

Conversation

erijo
Copy link
Collaborator

@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.

2 participants