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

Synchronous openRepository Fails #61

Closed
djrenren opened this issue Jul 19, 2013 · 8 comments
Closed

Synchronous openRepository Fails #61

djrenren opened this issue Jul 19, 2013 · 8 comments

Comments

@djrenren
Copy link

I try opening a repository like so:

var repo =  gitteh.openRepository(".git");

and receive:

node_modules/gitteh/lib/args.js:33
     throw new Error("Not enough arguments.");
Error: Not enough arguments.
@mildsunrise
Copy link
Contributor

Starting with 0.17.1, synchronous operations are now disabled.
For more info, see #51.

@mildsunrise
Copy link
Contributor

@samcday [duplicate of #51] [close request]

@samcday
Copy link
Contributor

samcday commented Jul 19, 2013

@jmrenner, what @jmendeth said ;) Synchronous operations aren't just disabled, they were never re-implemented when I did a big refactor of the codebase for 0.17. I decided not to implement synchronous behavior again as it makes the codebase much more complex. Apologies for any inconvenience this might have caused you!

@samcday samcday closed this as completed Jul 19, 2013
@mildsunrise
Copy link
Contributor

@samcday That remembers me, when compatibility is broken in such a way,
the major version should be bumped. So we should have 1.0.0 now.

@djrenren
Copy link
Author

Awesome, no problem! We should probably update the docs though.

@mildsunrise
Copy link
Contributor

@jmrenner Yeah, that's #53.

@samcday
Copy link
Contributor

samcday commented Jul 20, 2013

@jmendeth technically we're allowed to get away with breaking changes between "minor" releases in semver when the major version is still 0. I really wanna keep gitteh versioning tracking libgit2, as it feels more intuitive (at least to me)

Sent from my iPhone

On 20/07/2013, at 18:34, "Xavier Mendez" [email protected] wrote:

@jmrenner Yeah, that's #53.


Reply to this email directly or view it on GitHub.

@mildsunrise
Copy link
Contributor

👍 didn't know that.

2013/7/20 Sam [email protected]

@jmendeth technically we're allowed to get away with breaking changes
between "minor" releases in semver when the major version is still 0. I
really wanna keep gitteh versioning tracking libgit2, as it feels more
intuitive (at least to me)

Sent from my iPhone

On 20/07/2013, at 18:34, "Xavier Mendez" [email protected]
wrote:

@jmrenner Yeah, that's #53.


Reply to this email directly or view it on GitHub.


Reply to this email directly or view it on GitHubhttps://github.com//issues/61#issuecomment-21292700
.

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

No branches or pull requests

3 participants