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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Meroxa CLI #1377

Merged
merged 2 commits into from Aug 2, 2021
Merged

Add Meroxa CLI #1377

merged 2 commits into from Aug 2, 2021

Conversation

raulb
Copy link
Contributor

@raulb raulb commented Apr 16, 2021

馃憢,

I'm reaching out in behalf of Meroxa as one of its developers. We've been using Cobra since our first initial commit back from January 2020, and we recently launched our Platform to the public.

We continue considering using Cobra as our CLI framework and are actively working on it, and it'd be really nice to be featured as one of the projects using Cobra in the wild.

Thank you!

@CLAassistant
Copy link

CLAassistant commented Apr 16, 2021

CLA assistant check
All committers have signed the CLA.

@raulb
Copy link
Contributor Author

raulb commented Apr 16, 2021

Looks like the MINGW64 check will be be solved by #1366 so I'll rebase and will re-run CI once it gets merged.

Copy link
Collaborator

@marckhouzam marckhouzam left a comment

Easy one. /cc @jpmcb

raulb added 2 commits May 5, 2021
馃憢,

I'm reaching out in behalf of [Meroxa](https://meroxa.com/) as one of its developers. We've been using [Cobra](https://github.com/spf13/cobra) since our first [initial commit](meroxa/cli@a3301a0) back from January 2020, and we recently [launched our Platform](https://techcrunch.com/2021/04/13/meroxa-raises-15m-series-a-for-its-real-time-data-platform/) to the public.

We continue considering using Cobra as our CLI framework and are [actively](https://github.com/meroxa/cli/releases) working on it, and it'd be really nice to be featured as one of the projects using Cobra in the wild.

Thank you!
@raulb
Copy link
Contributor Author

raulb commented May 5, 2021

@marckhouzam @jpmcb Just updated with latest from master and all checks are passing now. Looks like this should be good to go! Thank you 馃檹 .

@raulb
Copy link
Contributor Author

raulb commented Jun 1, 2021

@marckhouzam @jpmcb 馃憢 wondering if someone else could help out shipping this easy one. Thank you!

@umarcor umarcor mentioned this pull request Jul 11, 2021
@github-actions
Copy link

github-actions bot commented Aug 1, 2021

This PR is being marked as stale due to a long period of inactivity

@raulb
Copy link
Contributor Author

raulb commented Aug 2, 2021

@marckhouzam @jpmcb This was marked as stale now even after being approved 馃槩

jpmcb
jpmcb approved these changes Aug 2, 2021
Copy link
Collaborator

@jpmcb jpmcb left a comment

@raulb Sorry for letting this go stale! Thanks for this!! LGTM 馃殌

@jpmcb jpmcb merged commit 56060d1 into spf13:master Aug 2, 2021
3 checks passed
@raulb raulb deleted the add-meroxa-cli branch Aug 2, 2021
@raulb
Copy link
Contributor Author

raulb commented Aug 2, 2021

@jpmcb no worries! Thank you for bringing it home! 馃嵒

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.

None yet

4 participants