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

Document the client-defined dunders #51227

Merged

Conversation

@astronouth7303
Copy link
Contributor

commented Jan 17, 2019

What does this PR do?

Document the dunder values added by SyncClientMixin.

This is a doc-only change.

What issues does this PR fix or reference?

None

@astronouth7303 astronouth7303 requested a review from saltstack/team-core as a code owner Jan 17, 2019
@astronouth7303

This comment has been minimized.

Copy link
Contributor Author

commented Jan 29, 2019

Updates?

@thatch45 thatch45 merged commit f511ad2 into saltstack:2017.7 Jan 29, 2019
3 of 10 checks passed
3 of 10 checks passed
jenkins/pr/lint Python lint test has failed
Details
jenkins/pr/py2-centos-7 running py2-centos-7...
Details
jenkins/pr/py2-ubuntu-1604 running py2-ubuntu-1604...
Details
jenkins/pr/py2-windows-2016 running py2-windows-2016...
Details
jenkins/pr/py3-centos-7 running py3-centos-7...
Details
jenkins/pr/py3-ubuntu-1604 running py3-ubuntu-1604...
Details
jenkins/pr/py3-windows-2016 running py3-windows-2016...
Details
WIP Ready for review
Details
continuous-integration/jenkins/pr-merge This commit looks good
Details
jenkins/pr/docs The docs job has passed
Details
@thatch45

This comment has been minimized.

Copy link
Member

commented Jan 29, 2019

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.