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

Removing islandora-connector-broadcast #56

Closed
wants to merge 1 commit into from

Conversation

dannylamb
Copy link
Contributor

GitHub Issue: Resolves Islandora/documentation#974

What does this Pull Request do?

Removes islandora-connector-broadcast since we've switched to pumping directly to queues from Drupal.

How should this be tested?

./gradlew build should run successfully and never mention islandora-connector-broadcast. The features file that gets written out to karaf/build/resources/features.xml should not contain a islandora-connector-broadcast entry.

Interested parties

@Islandora-CLAW/committers

@codecov
Copy link

codecov bot commented Nov 9, 2018

Codecov Report

Merging #56 into master will increase coverage by 4.35%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##             master      #56      +/-   ##
============================================
+ Coverage     80.36%   84.71%   +4.35%     
  Complexity       93       93              
============================================
  Files            17       16       -1     
  Lines           331      314      -17     
  Branches          1        1              
============================================
  Hits            266      266              
+ Misses           64       47      -17     
  Partials          1        1

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3a24020...aac9d65. Read the comment docs.

@dannylamb
Copy link
Contributor Author

Superseded by a4e7341

@dannylamb dannylamb closed this Mar 7, 2019
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.

1 participant