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

NOISSUE - Fix title #2

Merged
merged 1 commit into from
Mar 3, 2021
Merged

NOISSUE - Fix title #2

merged 1 commit into from
Mar 3, 2021

Conversation

darkodraskovic
Copy link
Contributor

Signed-off-by: Darko Draskovic darko.draskovic@gmail.com

Signed-off-by: Darko Draskovic <darko.draskovic@gmail.com>
Copy link
Contributor

@manuio manuio left a comment

Choose a reason for hiding this comment

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

LGTM

@nmarcetic nmarcetic merged commit 88c1d58 into mainflux:master Mar 3, 2021
Copy link
Member

@drasko drasko left a comment

Choose a reason for hiding this comment

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

@nmarcetic please do not merge when you see unresolved comment.

I demanded that title is corrected to Mfxkit - Mainflux Starter Kit, and that we remove lines Mfx-kit and mfx-kit - please use without dash in the middle.

@darkodraskovic can you please correct and send new PR?


## How-to

Copy `mfxkit` directory to the `mainflux` root directory, e.g. `~/go/src/github.com/mainflux/mainflux/`. Copy `cmd/mfxkit` directory to `mainflux/cmd` directory.
Copy `mfx-kit` directory to the `mainflux` root directory, e.g. `~/go/src/github.com/mainflux/mainflux/`. Copy `cmd/mfxkit` directory to `mainflux/cmd` directory.
Copy link
Member

Choose a reason for hiding this comment

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

Ah, I see now this is service / gopackage name. Sorry then, maybe we can keep.old name - I prefer one word mfxkit.

Maybe just add in title Mfxkit - Mainflux Starter Kit, and that's all.

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