Skip to content

Conversation

kiliit
Copy link

@kiliit kiliit commented Nov 26, 2015

Some extensions of the api

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What does this struct do? Seems not using it anywhere?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh. I didn't create the client method in the gogs client.

The DiffFile struct is used in gogs.

I tested the service method from the java lib and forgot the client method in gogs-client

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK... should you remove it or...?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No. It is used by Gogs, and is needed for my gogs pull request. See the DiffRange function in the that pull request.

This is also why the gogs pull request has dependency to kiliit/go-gogs-client. If it hasn't then compile fails.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry I don't get it...

Gogs has these structs itself, why need import from this package?

@unknwon
Copy link
Member

unknwon commented Jan 2, 2016

Hi, can you resolve conflicts?

@unknwon unknwon closed this Jan 15, 2016
guyzmo pushed a commit to guyzmo/go-gogs-client that referenced this pull request May 5, 2017
WIP: fix most of all the lint errors
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

2 participants