Skip to content

Update of the code with the update of some of the dependencies #558

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

Closed
wants to merge 1 commit into from

Conversation

mmoqui
Copy link
Member

@mmoqui mmoqui commented Oct 26, 2017

Don't forget to merge before:

Don't forget to merge then:

With the update of some of the dependencies and the removal of some of very old
dependencies, some codes have been refactored:

  • The very old informa library to produce or to consume RSS feeds is now
    replaced by the last version of Rome. All the codes that used informa are now
    refactored to use Rome instead of informa. This is mainly the case of the
    application RssAggregator.

  • Update the code of Gallery that uses the Apache HttpClient library with the
    new version of this library. In this new version, HttpClient provides
    Closeable classes for HttpClient, HttpResponse and so one. We use them now.

…f very old

dependencies, some codes have been refactored:
- The very old informa library to produce or to consume RSS feeds is now
  replaced by the last version of Rome. All the codes that used informa are now
  refactored to use Rome instead of informa. This is mainly the case of the
  application RssAggregator.

- Update the code of Gallery that uses the Apache HttpClient library with the
  new version of this library. In this new version, HttpClient provides
  Closeable classes for HttpClient, HttpResponse and so one. We use them now.
@SilverYoCha
Copy link
Member

Merged.
Closed manually because of manual git operations performed during integration.

@SilverYoCha SilverYoCha closed this Nov 7, 2017
@mmoqui mmoqui deleted the dependencies-update branch November 27, 2017 14:29
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.

2 participants