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

Fix respository (=> repository) typo in sls_build #46997

Merged
merged 1 commit into from Apr 11, 2018

Conversation

LukeCarrier
Copy link
Contributor

What does this PR do?

This typo looks to have been introduced with 64aa4fb and would affect anyone relying on the deprecated "name" attribute.

What issues does this PR fix or reference?

Previous Behavior

The name attribute would not have been treated as the repository value.

New Behavior

The name attribute now provides the value of the repository value, if present.

Tests written?

Yes/No

Commits signed with GPG?

No

Please review Salt's Contributing Guide for best practices.

See GitHub's page on GPG signing for more information about signing commits with GPG.

This typo looks to have been introduced with 64aa4fb and would affect
anyone relying on the deprecated "name" attribute.
Copy link
Contributor

@rallytime rallytime left a comment

Choose a reason for hiding this comment

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

Welcome to @LukeCarrier and thank you for fixing this.

@rallytime rallytime added the bugfix-bckport will be be back-ported to an older release branch by creating a PR against that branch label Apr 10, 2018
@rallytime rallytime merged commit 62d49e8 into saltstack:develop Apr 11, 2018
@rallytime rallytime added ZZZ[Done]-back-ported-bf RETIRED The pull request has been back-ported to an older branch. and removed bugfix-bckport will be be back-ported to an older release branch by creating a PR against that branch labels Apr 11, 2018
rallytime pushed a commit that referenced this pull request Apr 12, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ZZZ[Done]-back-ported-bf RETIRED The pull request has been back-ported to an older branch.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants