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

juniper modules/states docstring, parameter name fixes #57281

Merged
merged 20 commits into from
May 18, 2020

Conversation

vnitinv
Copy link
Contributor

@vnitinv vnitinv commented May 14, 2020

What does this PR do?

Fix Juniper#141 & Juniper#142

What issues does this PR fix or reference?

Fixes: Minor doc strings and parameter names

Merge requirements satisfied?

[NOTICE] Bug fixes or features added to Salt require tests.

Commits signed with GPG?

Yes/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.

@vnitinv vnitinv requested a review from a team as a code owner May 14, 2020 15:53
@ghost ghost requested review from DmitryKuzmenko and removed request for a team May 14, 2020 15:54
@vnitinv
Copy link
Contributor Author

vnitinv commented May 14, 2020

@dmurphy18 Please help us with your review. I am writing UT for these changes. Started pull request to keep you posted about this change.

salt/states/junos.py Outdated Show resolved Hide resolved
salt/states/junos.py Outdated Show resolved Hide resolved
salt/states/junos.py Outdated Show resolved Hide resolved
salt/states/junos.py Outdated Show resolved Hide resolved
salt/states/junos.py Outdated Show resolved Hide resolved
salt/states/junos.py Outdated Show resolved Hide resolved
salt/states/junos.py Outdated Show resolved Hide resolved
tests/unit/proxy/test_junos.py Outdated Show resolved Hide resolved
salt/modules/junos.py Outdated Show resolved Hide resolved
salt/modules/junos.py Outdated Show resolved Hide resolved
@vnitinv vnitinv requested a review from dmurphy18 May 18, 2020 15:25
dmurphy18
dmurphy18 previously approved these changes May 18, 2020
@dwoz dwoz merged commit 1bbb8ef into saltstack:master May 18, 2020
@sagetherage sagetherage added the ZRelease-Sodium retired label label May 20, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ZRelease-Sodium retired label
Projects
None yet
Development

Successfully merging this pull request may close these issues.

handle localhost session
4 participants