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

readme.md clarifications, examples and typos #930

Merged
merged 4 commits into from
May 29, 2017
Merged

readme.md clarifications, examples and typos #930

merged 4 commits into from
May 29, 2017

Conversation

sycnus
Copy link
Contributor

@sycnus sycnus commented May 25, 2017

Tried to keep it concise, I found it a little incoherent before.

@coveralls
Copy link

coveralls commented May 25, 2017

Coverage Status

Coverage remained the same at 93.091% when pulling e80a84d on mwkarttu:readme-clarification into 14c8a3f on vpulim:master.

@herom
Copy link
Contributor

herom commented May 29, 2017

thanks a lot @mwkarttu - this makes it more read- & understandable indeed 👍

@herom
Copy link
Contributor

herom commented May 29, 2017

I try to rebase & merge your PR so that the test won't be ran against the legacy node.js versions which break tests because of npm and node.js version mismatch...

Could you please rebase your branch in order to make the tests pass? this would be awesome 😸

sycnus and others added 4 commits May 29, 2017 12:54
Tried to keep it concise, I found it a little incoherent before.
extended a few examples for clarity
@sycnus
Copy link
Contributor Author

sycnus commented May 29, 2017

Rebased and added a few more things.

@sycnus sycnus changed the title clarified WSSecurity example in readme.md readme.md clarifications, examples and typos May 29, 2017
@coveralls
Copy link

coveralls commented May 29, 2017

Coverage Status

Coverage remained the same at 93.091% when pulling c87cc5d on mwkarttu:readme-clarification into 18c2790 on vpulim:master.

@herom
Copy link
Contributor

herom commented May 29, 2017

thanks a ton @mwkarttu 👍

@herom herom merged commit 358c611 into vpulim:master May 29, 2017
jcald1 pushed a commit to jcald1/node-soap that referenced this pull request Jun 8, 2017
* clarified WSSecurity example in readme.md

Tried to keep it concise, I found it a little incoherent before.

* typos and "linting"

* reorganized wsdlOptions slightly
extended a few examples for clarity

* added debugging proxy example
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants