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

(GH-332) Ensure PuppetStrings is loaded for tasks #333

Merged
merged 1 commit into from
Oct 25, 2022

Conversation

chelnak
Copy link
Contributor

@chelnak chelnak commented Oct 25, 2022

Prior to this PR users would receive an Undefined Constant error when attempting to generate a reference file via the provided Rake tasks.

This was caused by the compact testing style for modules added in v3.0.

In this case the parent, PuppetStrings is not properly loaded before Tasks. As a result the error above is caused.

This PR adds a new module declaration to tasks.rb that will ensure that the PuppetStrings is properly loaded.

@chelnak chelnak added the bug label Oct 25, 2022
@chelnak chelnak self-assigned this Oct 25, 2022
@chelnak chelnak requested a review from a team as a code owner October 25, 2022 14:59
Prior to this commit users would receive an Undefined Constant error
when attempting to generate a reference file via the provided Rake
tasks.

This was caused by the compact testing style for modules added in v3.0.

In this case the parent, PuppetStrings is not properly loaded before
Tasks. As a result the error above is caused.

This commit adds a new module declaration to tasks.rb that will ensure
that the PuppetStrings is properly loaded.
@chelnak chelnak force-pushed the GH-332-ensure_module_is_loaded branch from 0c860b4 to 0292153 Compare October 25, 2022 15:00
@LukasAud LukasAud merged commit 55e3091 into main Oct 25, 2022
@LukasAud LukasAud deleted the GH-332-ensure_module_is_loaded branch October 25, 2022 17:52
@chelnak chelnak linked an issue Oct 26, 2022 that may be closed by this pull request
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.

Undefined Constant when running via Rake
2 participants