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

AMQP connector #42

Merged
merged 6 commits into from
Aug 16, 2016
Merged

Conversation

johanandren
Copy link
Member

Refs #41


lazy val amqp = project.
enablePlugins(AutomateHeaderPlugin)
.configs(IntegrationTest)
Copy link
Member Author

Choose a reason for hiding this comment

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

How can I add a config in amqp/build.sbt? Couldn't figure it out.

Copy link
Contributor

Choose a reason for hiding this comment

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

Was not sure as well. After a bit of poking around, it looks like we can set it in amqp/build.sbt by adding:

lazy val amqp = (project in file(".")).
  configs(IntegrationTest)
  enablePlugins(AutomateHeaderPlugin)

Enabling plugins work there as well. So we can leave ~/build.sbt with only

lazy val amqp = project

@patriknw
Copy link
Member

LGTM, great work @johanandren

@@ -8,5 +8,6 @@ cache:
directories:
- $HOME/.ivy2/cache
- $HOME/.sbt/boot
sudo: required
Copy link
Contributor

Choose a reason for hiding this comment

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

yup

@johanandren
Copy link
Member Author

Yay, integration tests running on travis, and passing.

@johanandren johanandren changed the title [wip] AMQP connector AMQP connector Aug 16, 2016
@johanandren johanandren force-pushed the wip-amqp-connector-johanandren branch from 5834877 to e99c926 Compare August 16, 2016 09:46
@2m
Copy link
Contributor

2m commented Aug 16, 2016

LGTM

@johanandren johanandren merged commit 6bb239e into akka:master Aug 16, 2016
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.

4 participants