Skip to content
This repository has been archived by the owner on Aug 18, 2020. It is now read-only.

Using LATEST API-X #49

Merged

Conversation

dannylamb
Copy link
Contributor

GitHub Issue: Islandora/documentation#666

Blocks

What does this Pull Request do?

A brief description of what the intended result of the PR will be and/or what problem it solves.

What's new?

Pulls in SNAPSHOT builds of API-X.

How should this be tested?

  • Pull in these changes
  • vagrant up
  • /opt/karaf/bin/client
  • la | grep api-x

You should see 0.3.0-SNAPSHOT for all the versions. Mine looked like this:

karaf@root()> la | grep api-x
136 | Active   |  80 | 0.3.0.SNAPSHOT     | fcrepo-api-x-binding
137 | Active   |  80 | 0.3.0.SNAPSHOT     | fcrepo-api-x-jena
138 | Active   |  80 | 0.3.0.SNAPSHOT     | fcrepo-api-x-listener
139 | Active   |  80 | 0.3.0.SNAPSHOT     | fcrepo-api-x-loader
140 | Active   |  80 | 0.3.0.SNAPSHOT     | fcrepo-api-x-model
141 | Active   |  80 | 0.3.0.SNAPSHOT     | fcrepo-api-x-registry
142 | Active   |  80 | 0.3.0.SNAPSHOT     | fcrepo-api-x-routing

Interested parties

@Islandora-CLAW/committers @ajs6f

Copy link
Contributor

@ajs6f ajs6f left a comment

Choose a reason for hiding this comment

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

👍

@ajs6f
Copy link
Contributor

ajs6f commented Jun 27, 2017

@dannylamb Is this change (from a policy POV) specific to API-X, or are we okay now with pulling snapshot/latest versions of all dependencies (NOT releasing with them, of course, just developing with them)?

@dannylamb
Copy link
Contributor Author

@ajs6f This is a one-off situation, but we can evaluate other dependencies on a case-by-case basis. I have no real issues with SNAPSHOT dependencies until we release.

@ajs6f
Copy link
Contributor

ajs6f commented Jun 27, 2017

Sounds good @dannylamb. Case-by-case = agile.

@ruebot ruebot merged commit 4662c37 into islandora-deprecated:master Jun 27, 2017
@ruebot
Copy link
Contributor

ruebot commented Jun 27, 2017

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants