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

Connector uses unsupported API (#47) #50

Conversation

PhilMFischer
Copy link
Contributor

  • Fixed compile issues due to changes in java-gitlab-api

  • Prepared for Including local Snapshot build of java-gitlab-api 4.0.1 with fixes of GitlabAPI.getNotes(...) call creating non api v4 compliant URL (#283) timols/java-gitlab-api#284

  • Added snapshots to gitignore file

  • Replaced external libraries as needed by java gitlab api 4.0.x

  • Added jackson-core 2.5.3

  • Added jackson-databind 2.5.3

  • Added jackson-annotations 2.5.3

  • Added commons-io 2.4

  • Updated Licensing information to show about and added third party licenses as much as they are available. Now following eclipse style handling the licenses in plugins.

  • Updated plugin license information for java-gitlab-api

- Updated Licensing information to show about and added third party
licenses as much as they are available
- Updated plugin license information for java-gitlab-api

- Replaced external libraries as needed by java gitlab api 4.0.x
- Added jackson-core 2.5.3
- Added jackson-databind 2.5.3
- Added jackson-annotations 2.5.3
- Added commons-io 2.4

- Included local Snapshot build of java-gitlab-api 4.0.1
- Added snapshots to gitignore file

- Fixed compile issues due to changes in java-gitlab-api
<p>July, 2016</p>
<h3>License</h3>

<p> Unless otherwise indicated below, all content of this plugin &quot;Content&quot; is provided to you under the terms
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@pweingardt usually if you look into the about.html of eclipse plugins. The author is emntiones as well in a way such as "The content of this plugin is made available by pweingart". Was not sure if you would like it or not. Please check if you like it or not.

@samrocketman
Copy link

IMO the license bit probably should have been a separate PR. You've mixed a couple issues in one PR so it became pretty big (24 files is not a small amount).

It's not clear if the license files were copied from the repository license or elsewhere at a glance.

Copy link

@maba001 maba001 left a comment

Choose a reason for hiding this comment

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

I came to the exact same change results (apart from the doc / license enhancements). In my opinion this looks okay to be merged.

@vincowl
Copy link

vincowl commented Sep 1, 2018

@PhilMFischer How would it be possible to test it locally ? I mean deploy it in a Photon Eclipse installation for instance. Sadly, I don't know much about Eclipse plugins...

@pweingardt
Copy link
Owner

I will look into this pull request on the weekend.

@fogs
Copy link

fogs commented Oct 13, 2018

Would it help to integrate this PR if we split it into two separate ones; one for API v4 and one for the licensing? Or get you a beer or a coffee? ;)

@pweingardt pweingardt merged commit 0100b6f into pweingardt:master Oct 14, 2018
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.

6 participants