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

Creating issues? #8

Closed
IainNZ opened this issue Feb 26, 2014 · 11 comments
Closed

Creating issues? #8

IainNZ opened this issue Feb 26, 2014 · 11 comments

Comments

@IainNZ
Copy link
Contributor

IainNZ commented Feb 26, 2014

First of all, great work, love the package.

I wanted to know if creating issues through the API is coming any time soon? If not I could try to make a PR.

@WestleyArgentum
Copy link
Collaborator

Ah, I've looked at the api but not written anything. I could give it a shot tonight, or you're more than welcome to mess with it if it's blocking you

WestleyArgentum added a commit that referenced this issue Feb 28, 2014
@WestleyArgentum
Copy link
Collaborator

Ok, still a little ways to go, but I got in issue(user, repo, issue_num) and create_issue(user, repo, title; body, assignee, milestone, labels)

Hope that helps! I'll keep chipping away at things, and pull requests are more than welcome

@WestleyArgentum
Copy link
Collaborator

Ok, you should now be able to create issues, get them individually or by repo, and edit / close them. I don't have docs yet, but you should be able to see http://developer.github.com/v3/issues/ for parameter values (and feel free to just ask).

I'm going to leave this open until commenting, docs, etc are done, but is there anything else issue-related that you need soon?

@IainNZ
Copy link
Contributor Author

IainNZ commented Mar 2, 2014

No, thats great thanks!

@IainNZ
Copy link
Contributor Author

IainNZ commented May 14, 2014

I'm about to start using this to auto-file issues from PackageEvaluator stuff, so that should really give it a could test out.

@IainNZ
Copy link
Contributor Author

IainNZ commented May 14, 2014

create_issue("JuliaOpt", "JuMP.jl", "Test Issue",
    auth=my_auth,
    body="This issue was generated using GitHub.jl")

causes

WARNING: convert{T}(p::Type{Ptr{T}},a::Array) is deprecated, use convert(p,pointer(a)) instead.
 in Parser at /home/idunning/.julia/v0.3/HttpParser/src/HttpParser.jl:48
 in ResponseParser at /home/idunning/.julia/v0.3/Requests/src/Requests.jl:66
 in process_response at /home/idunning/.julia/v0.3/Requests/src/Requests.jl:225
 in do_request at /home/idunning/.julia/v0.3/Requests/src/Requests.jl:562
 in get at /home/idunning/.julia/v0.3/Requests/src/Requests.jl:576
 in authenticate at /home/idunning/.julia/v0.3/GitHub/src/auth.jl:41
 in include at boot.jl:244
 in include_from_node1 at loading.jl:128
ERROR: no method convert(Type{String}, Dict{Any,Any})
 in default_request at /home/idunning/.julia/v0.3/Requests/src/Requests.jl:39
 in default_request at /home/idunning/.julia/v0.3/Requests/src/Requests.jl:50
 in open_stream at /home/idunning/.julia/v0.3/Requests/src/Requests.jl:202
 in do_request at /home/idunning/.julia/v0.3/Requests/src/Requests.jl:562
 in post at /home/idunning/.julia/v0.3/Requests/src/Requests.jl:576
 in create_issue at /home/idunning/.julia/v0.3/GitHub/src/issues.jl:138
 in create_issue at /home/idunning/.julia/v0.3/GitHub/src/issues.jl:116
 in include at boot.jl:244
 in include_from_node1 at loading.jl:128

@WestleyArgentum
Copy link
Collaborator

Ah - I think I can fix those pretty easily. Shooting from the hip here, but I remember that Requests.jl changed recently, I might need to update.

@IainNZ
Copy link
Contributor Author

IainNZ commented May 14, 2014

Yeah, I'm guessing thats it

@WestleyArgentum
Copy link
Collaborator

I think I've fixed this on master - let me know if you run into any more trouble

@WestleyArgentum
Copy link
Collaborator

I think this was resolved? Did you end up using this in PackageEvaluator to file issues?

This could be better documented still, but I'm closing in favor of #28

@IainNZ
Copy link
Contributor Author

IainNZ commented Feb 21, 2015

I ended up doing it manually
https://github.com/IainNZ/packages.julialang.org/blob/395ff8db8bf3b2fdab411db23f76bb42d3e43dd3/script/file_issues.jl#L93-L101
but if/when I refactor, I'll shift over.

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

2 participants