Skip to content

Configure youtube stub for bok-choy - #3440

Merged
muhammad-ammar merged 1 commit into
masterfrom
ammar/config-youtube-stub-for-bok-choy
Apr 28, 2014
Merged

Configure youtube stub for bok-choy#3440
muhammad-ammar merged 1 commit into
masterfrom
ammar/config-youtube-stub-for-bok-choy

Conversation

@muhammad-ammar

Copy link
Copy Markdown
Contributor

Configurations to run youtub stub server for bok-choy tests.

@muhammad-ammar

Copy link
Copy Markdown
Contributor Author

@jzoldak Please Review.

Comment thread cms/envs/bok_choy.py Outdated

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

bok_choy imports * from .aws and .aws imports * from .common so this is not necessary

@jzoldak

jzoldak commented Apr 25, 2014

Copy link
Copy Markdown
Contributor

Looks good to me 👍
@valera-rozuvan can you pls re-review?

@valera-rozuvan

Copy link
Copy Markdown
Contributor

@jzoldak I have re-reviewed this PR. @muhammad-ammar 👍 Good to merge!

muhammad-ammar added a commit that referenced this pull request Apr 28, 2014
@muhammad-ammar
muhammad-ammar merged commit dfa45ea into master Apr 28, 2014
@muhammad-ammar
muhammad-ammar deleted the ammar/config-youtube-stub-for-bok-choy branch April 28, 2014 12:32
@muhammad-ammar
muhammad-ammar restored the ammar/config-youtube-stub-for-bok-choy branch May 2, 2014 20:37
@benpatterson
benpatterson deleted the ammar/config-youtube-stub-for-bok-choy branch January 7, 2015 13:12
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.

5 participants