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

ffead-cpp-apache2 addition #2161

Closed
wants to merge 4 commits into
base: round-14
from

Conversation

Projects
None yet
3 participants
@nbrady-techempower
Member

nbrady-techempower commented Jul 12, 2016

This updates #2061 ffead update and apache test addition

@nbrady-techempower

This comment has been minimized.

Show comment
Hide comment
@nbrady-techempower

nbrady-techempower Jul 14, 2016

Member

@sumeetchhetri anything i may have missed here?

Member

nbrady-techempower commented Jul 14, 2016

@sumeetchhetri anything i may have missed here?

@sumeetchhetri

This comment has been minimized.

Show comment
Hide comment
@sumeetchhetri

sumeetchhetri Jul 15, 2016

Contributor

Looks all set Nate!!

Contributor

sumeetchhetri commented Jul 15, 2016

Looks all set Nate!!

@nbrady-techempower

This comment has been minimized.

Show comment
Hide comment
@nbrady-techempower

nbrady-techempower Jul 18, 2016

Member

@sumeetchhetri it's not passing travis, do you mind taking a look when you have a moment

Member

nbrady-techempower commented Jul 18, 2016

@sumeetchhetri it's not passing travis, do you mind taking a look when you have a moment

@sumeetchhetri sumeetchhetri referenced this pull request Jul 27, 2016

Merged

Sc ffead cpp #2173

@sumeetchhetri

This comment has been minimized.

Show comment
Hide comment
@sumeetchhetri

sumeetchhetri Jul 27, 2016

Contributor

Nate, unable to raise a PR against your forked repository, hence am raising another new PR directly against the base repo. Please close this one, I have fixed the issue in a new PR.

Contributor

sumeetchhetri commented Jul 27, 2016

Nate, unable to raise a PR against your forked repository, hence am raising another new PR directly against the base repo. Please close this one, I have fixed the issue in a new PR.

@sumeetchhetri

This comment has been minimized.

Show comment
Hide comment
@sumeetchhetri
Contributor

sumeetchhetri commented Jul 27, 2016

@knewmanTE

This comment has been minimized.

Show comment
Hide comment
@knewmanTE

knewmanTE Aug 6, 2016

Contributor

closing in favor it #2173

Contributor

knewmanTE commented Aug 6, 2016

closing in favor it #2173

@knewmanTE knewmanTE closed this Aug 6, 2016

@nbrady-techempower nbrady-techempower deleted the nbrady-techempower:ffead-cpp branch Aug 28, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment