StackStorm-Exchange / exchange-incubator

Submit your StackStorm integration and automation packs here.
12 stars 61 forks source link

PR: to raise request for a pack submission from IBM #162

Closed Abhay-Rastogi closed 3 years ago

Abhay-Rastogi commented 3 years ago

Hi Team

this PR is initialisation of pack submission to StackStorm Community..

Name of pack : Monitor Mqtt

Please let us know for any modifications.

Abhay-Rastogi commented 3 years ago

Hi StackStorm Team

Please check why this PR request failed.

-Abhay

blag commented 3 years ago

This PR is currently failing CI checks because it is missing a LICENSE file.

All future packs on StackStorm Exchange must be licensed under the Apache 2.0 license and contain a copy of the license text in a file named LICENSE at the root of the repository.

See https://github.com/StackStorm-Exchange/stackstorm-device42/blob/master/LICENSE for an example. You should be able to copy and paste that file into your repository to get this PR to pass that CI check.

carlosibm commented 3 years ago

The team has added the license file. Can you please review the pull request and approve it? Also can you please provide a rough estimate for when the pull request will be approved? Thanks for your help.

Abhay-Rastogi commented 3 years ago

Hi @nmaludy

I have added License file, now it is failing with

/home/circleci/repo/actions/publish.py:13:41: E226 missing whitespace around arithmetic operator

Please check

image

Abhay-Rastogi commented 3 years ago

Hi @nmaludy

please review the request.

Abhay-Rastogi commented 3 years ago

Hi @nmaludy

Please review i have done up with changes.

nmaludy commented 3 years ago

@Abhay-Rastogi i'm working on moving this into its own repo, but hit a small snag, i'll let you know when it's ready

Abhay-Rastogi commented 3 years ago

sure @nmaludy

Abhay-Rastogi commented 3 years ago

Hi @nmaludy

Any update ?

-Abhay

carlosibm commented 3 years ago

@nmaludy We have several clients and partners that would like to use these packs. Could you please give us an update when the pull request will be approved. Thanks.

nmaludy commented 3 years ago

It is ready here: https://github.com/StackStorm-Exchange/stackstorm-monitor_mqtt