[2016.11.2/napalm] Better error message when NotImplementedError raised #38682

Merged
merged 1 commit into from Jan 15, 2017

Projects

None yet

2 participants

@mirceaulinic
Contributor

What does this PR do?

Yup, yet another PR... apologies.
This one only brings some makeup as users complained that the error message is not clear enough when a certain feature is not implemented in the underlying library.

@mirceaulinic mirceaulinic Better error message when NotImplementedError raised
f847639
@mirceaulinic mirceaulinic changed the title from Better error message when NotImplementedError raised to [napalm] Better error message when NotImplementedError raised Jan 11, 2017
@mirceaulinic mirceaulinic changed the title from [napalm] Better error message when NotImplementedError raised to [2016.11.2/napalm] Better error message when NotImplementedError raised Jan 11, 2017
@cachedout
Contributor

Go Go Jenkins!

@cachedout cachedout merged commit bf6d74c into saltstack:2016.11 Jan 15, 2017

3 of 6 checks passed

default Build finished.
Details
jenkins/PR/salt-pr-linode-ubuntu14-n Pull Requests » Salt Linode Ubuntu14.04 #7639 — FAILURE
Details
jenkins/PR/salt-pr-rs-cent7-n Pull Requests » Salt PR - RS CentOS 7 #7937 — FAILURE
Details
jenkins/PR/salt-pr-clone Pull Requests » Salt PR - Clone #7973 — SUCCESS
Details
jenkins/PR/salt-pr-docs-n Pull Requests » salt-pr-docs-n #3264 — SUCCESS
Details
jenkins/PR/salt-pr-lint-n Pull Requests » Salt PR - Code Lint #7790 — SUCCESS
Details
@cachedout
Contributor

Keep 'em coming, @mirceaulinic. These are great!

@mirceaulinic mirceaulinic deleted the cloudflare:NotImplementedError-MSG branch Jan 15, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment