Skip to content
This repository was archived by the owner on Apr 8, 2021. It is now read-only.

Comments

Replace Finatra 1.6 with finagle-http, fix quick start and examples#100

Open
travisbrown wants to merge 8 commits intostripe-archive:masterfrom
travisbrown:topic/de-finatrify
Open

Replace Finatra 1.6 with finagle-http, fix quick start and examples#100
travisbrown wants to merge 8 commits intostripe-archive:masterfrom
travisbrown:topic/de-finatrify

Conversation

@travisbrown
Copy link

This PR includes @eightysteele's changes from #95 but rebased and with a few little fixes for recent updates.

It also replaces brushfire-finatra with brushfire-finagle, which does more or less the same thing but without the dependency on the EOL-ed Finatra 1 (which won't be published for 2.12). Since this is just an example project (that wasn't compiling anyway) I figured a pretty major change wouldn't be a problem. We could also just delete the dead code, but it seems like a nice thing to have.

I've added an ./iris-srv example to the README, since it works again. I'd like to aggregate the module in the build, so that we know when it breaks, but because the latest Finagle version isn't built for 2.10 this causes problems for ++ and + (we could use sbt-doge to fix this but in my experience it's a pain and in this case probably isn't worth the effort).

More generally it seems like it'd be a good idea to pull all of the example code into a new brushfire-example module, so that stuff like local.Example isn't ending up in our published artifacts. If people agree I could do that in this PR, or in a follow-up.

@CLAassistant
Copy link

CLAassistant commented Aug 6, 2020

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
0 out of 2 committers have signed the CLA.

❌ eightysteele
❌ travisbrown
You have signed the CLA already but the status is still pending? Let us recheck it.

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