Skip to content

Report authentication errors #15

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

crystalneth
Copy link

Tests don't seem to work or be maintained, so no test...

@dscape
Copy link

dscape commented Jun 17, 2012

We normally don't take PR without issues so please feel free to re-open with tests.

If you have an issue with existing tests please feel free to submit an issue :)

Thanks for contributing!

@crystalneth
Copy link
Author

The existing tests do not run.

@crystalneth
Copy link
Author

The tests do not run, and the documentation and error messages are incorrect. The docs/message indicate a config file under 'data', but actually it looks for one under 'fixtures'. Even with that config file, the tests fail. That makes it difficult to test.

BTW the current NPM version is not working.

@indexzero
Copy link
Member

Sorry for the delay here! We've been waiting on all the issues / pull-requests for node-cloudfiles and node-cloudservers until we released pkgcloud: a new cross-provider unified IaaS library for node.js:

https://github.com/nodejitsu/pkgcloud
http://blog.nodejitsu.com/introducing-pkgcloud

We added a deprecation notice for this library, but would you consider reimplementing your pull-request for the Rackspace provider in pkgcloud? It would make you a HERO in my book 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants