ovh / python-ovh

Thin wrapper around OVH's APIs. Handles all the hard work including credential creation and requests signing.
https://pypi.org/project/ovh/
Other
278 stars 76 forks source link

Update US Endpoint #67

Closed tanji closed 5 years ago

tanji commented 5 years ago

The US endpoint Domain Name has changed. Reflecting this change in the endpoint mapping list.

By making a contribution to this project, I certify that:

The contribution was created in whole or in part by me and I have the right to submit it under the open source license indicated in the file; or The contribution is based upon previous work that, to the best of my knowledge, is covered under an appropriate open source License and I have the right under that license to submit that work with modifications, whether created in whole or in part by me, under the same open source license (unless I am permitted to submit under a different license), as indicated in the file; or The contribution was provided directly to me by some other person who certified (a), (b) or (c) and I have not modified it. The contribution is made free of any other party's intellectual property claims or rights. I understand and agree that this project and the contribution are public and that a record of the contribution (including all personal information I submit with it, including my sign-off) is maintained indefinitely and may be redistributed consistent with this project or the open source license(s) involved.

Signed-off-by: Guillaume Lefranc guillaume@signal18.io

rbeuque74 commented 5 years ago

Build is currently failed (see #68), in the meantime, I will merge this pull-request which is not responsible of broken build.

tanji commented 5 years ago

I already did this in the PR, is it insufficient? I don't think I can edit my commit without closing this PR

rbeuque74 commented 5 years ago

The "legal" text is not mandatory, you just need to sign-off the commit using the flag -s while commiting.

You can edit your commit without closing the pull-request.

git commit -a --amend --no-edit -s
git push origin patch-1 -f

While you are modifying your commit, could you modify the URL in the README.rst for api.ovhcloud.com -> api.us.ovhcloud.com ? If you can't do it, I will do it in another commit, don't worry ;)

rbeuque74 commented 5 years ago

Merged as of #70