Skip to content

Fix recipe filler for the case the variable doesn't contain short_name#2104

Merged
valeriupredoi merged 3 commits into
masterfrom
fix_recipe_filler
Mar 26, 2021
Merged

Fix recipe filler for the case the variable doesn't contain short_name#2104
valeriupredoi merged 3 commits into
masterfrom
fix_recipe_filler

Conversation

@valeriupredoi
Copy link
Copy Markdown
Contributor

@valeriupredoi valeriupredoi commented Mar 26, 2021

Closes #2095

@valeriupredoi valeriupredoi requested a review from aschurer March 26, 2021 15:11
@valeriupredoi valeriupredoi mentioned this pull request Mar 26, 2021
Closed
aschurer
aschurer approved these changes Mar 26, 2021
View reviewed changes
Copy link
Copy Markdown

@aschurer aschurer left a comment

Choose a reason for hiding this comment

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

I agree this will fix the bug.

@valeriupredoi
Copy link
Copy Markdown
Contributor Author

valeriupredoi commented Mar 26, 2021

cheers muchly @aschurer 🍺

@valeriupredoi valeriupredoi merged commit 988ae99 into master Mar 26, 2021
@valeriupredoi valeriupredoi deleted the fix_recipe_filler branch March 26, 2021 17:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@aschurer aschurer aschurer approved these changes

Assignees

No one assigned

Labels

bug

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Potential issue in recipe_filler

2 participants

@valeriupredoi @aschurer