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

Add warning about using "=" in file.line function #30163

Merged
merged 1 commit into from Jan 6, 2016

Conversation

Projects
None yet
2 participants
@rallytime
Contributor

rallytime commented Jan 5, 2016

Warnings for passing kwargs with equal signs as part of the value
exist in other file.x functions, but was missing for the file.line
execution module function. This adds the relevant warning.

Fixes #28923

Add warning about using "=" in file.line function
Warnings for passing kwargs with equal signs as part of the value
exist in other file.x functions, but was missing for the file.line
execution module function. This adds the relevant warning.

Fixes #28923

basepi added a commit that referenced this pull request Jan 6, 2016

Merge pull request #30163 from rallytime/fix-28923
Add warning about using "=" in file.line function

@basepi basepi merged commit 8324593 into saltstack:2015.8 Jan 6, 2016

2 of 5 checks passed

jenkins/salt-pr-linode-ubuntu14.04-n Salt PR - Linode Ubuntu 14.04 #3955 — FAILURE
Details
jenkins/salt-pr-rs-cent7-n Salt PR - RS CentOS 7 #11097 — FAILURE
Details
default Merged build started.
Details
jenkins/salt-pr-clone Salt PR - Clone Repository #12510 — SUCCESS
Details
jenkins/salt-pr-lint-n Salt PR - Code Lint #12209 — SUCCESS
Details
@basepi

This comment has been minimized.

Show comment
Hide comment
@basepi

basepi Jan 6, 2016

Collaborator

Looks great.

Collaborator

basepi commented Jan 6, 2016

Looks great.

@rallytime rallytime deleted the rallytime:fix-28923 branch Jan 6, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment