Skip to content

Stackstorm-nexus3 pack review request#138

Closed
sahilsk wants to merge 7 commits intoStackStorm-Exchange:masterfrom
AllForkedRepos:master
Closed

Stackstorm-nexus3 pack review request#138
sahilsk wants to merge 7 commits intoStackStorm-Exchange:masterfrom
AllForkedRepos:master

Conversation

@sahilsk
Copy link
Copy Markdown

@sahilsk sahilsk commented Feb 22, 2019

This is a pack for sontatype Nexus Repository.

Kindly help me finish this pack by reviewing it.

Copy link
Copy Markdown
Contributor

@blag blag left a comment

Choose a reason for hiding this comment

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

Looking very good so far. Only a few minor changes necessary, or maybe just comment why you implemented things a specific way.

Comment thread stackstorm-nexus3/README-test.md Outdated
Comment thread stackstorm-nexus3/README-test.md Outdated
Comment thread stackstorm-nexus3/README.md Outdated
Comment thread stackstorm-nexus3/README.md Outdated
Comment thread stackstorm-nexus3/README.md Outdated
Comment thread stackstorm-nexus3/actions/lib/base.py Outdated
Comment thread stackstorm-nexus3/actions/lib/base.py Outdated
Comment thread stackstorm-nexus3/config.schema.yaml Outdated
Comment thread stackstorm-nexus3/pack.yaml
Comment thread stackstorm-nexus3/tests/test_run.py Outdated
@blag
Copy link
Copy Markdown
Contributor

blag commented Feb 22, 2019

And I would like to see CI passing before promoting this. CI checks are currently failing:
https://github.com/StackStorm-Exchange/exchange-incubator/pull/138/checks?check_run_id=67194904

Copy link
Copy Markdown
Contributor

@blag blag left a comment

Choose a reason for hiding this comment

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

Looking good - since the one unresolved issue is in the README I'm approving this.

Comment thread stackstorm-nexus3/README.md Outdated
@blag
Copy link
Copy Markdown
Contributor

blag commented Feb 25, 2019

Everything is looking good, thank you for fixing the issues I pointed out. I'll be moving this to its own repository in a bit.

@blag
Copy link
Copy Markdown
Contributor

blag commented Feb 27, 2019

PR moved to its own repository: https://github.com/StackStorm-Exchange/stackstorm-nexus3

@blag blag closed this Feb 27, 2019
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.

2 participants