add playground to show sample code and modify README to explain how to b... #5

Merged
merged 6 commits into from Sep 5, 2014

Conversation

Projects
None yet
4 participants
@corinnekrych
Member

corinnekrych commented Aug 28, 2014

...uild it

@danbev

This comment has been minimized.

Show comment
Hide comment
@danbev

danbev Aug 28, 2014

Perhaps we could simple get the first joke using "http://api.icndb.com/jokes/1"?
And it might be nice to add an not in the README.md how to see the console output in the assistent editor.

I like the idea being able to quickly out a code example.

Perhaps we could simple get the first joke using "http://api.icndb.com/jokes/1"?
And it might be nice to add an not in the README.md how to see the console output in the assistent editor.

I like the idea being able to quickly out a code example.

@abstractj

This comment has been minimized.

Show comment
Hide comment
@abstractj

abstractj Sep 4, 2014

@danbev would you like to give a second try?

@corinnekrych any idea why travis is upset?

@danbev would you like to give a second try?

@corinnekrych any idea why travis is upset?

@danbev

This comment has been minimized.

Show comment
Hide comment
@danbev

danbev Sep 4, 2014

Member

@abstractj absolutely, will try now.

Member

danbev commented Sep 4, 2014

@abstractj absolutely, will try now.

@corinnekrych

This comment has been minimized.

Show comment
Hide comment
@corinnekrych

corinnekrych Sep 4, 2014

Member

@abstractj Travis is upset because he doesn't know Xcode6 yet and i think until it's official release we won't have it.

Member

corinnekrych commented Sep 4, 2014

@abstractj Travis is upset because he doesn't know Xcode6 yet and i think until it's official release we won't have it.

@abstractj

This comment has been minimized.

Show comment
Hide comment
@abstractj

abstractj Sep 4, 2014

@corinnekrych not sure if I'm saying something crazy. We have 2 options: disable travis for this project (I don't love it, but....) or make it build with Xcode 5.1 http://docs.travis-ci.com/user/osx-ci-environment/. Thoughts?

@corinnekrych not sure if I'm saying something crazy. We have 2 options: disable travis for this project (I don't love it, but....) or make it build with Xcode 5.1 http://docs.travis-ci.com/user/osx-ci-environment/. Thoughts?

@corinnekrych

This comment has been minimized.

Show comment
Hide comment
@corinnekrych

corinnekrych Sep 4, 2014

Member

@abstractj I don't think the second option is doable running Swift code does require Xcode6, so we're left with the first option: disable travis for now. Xcode6 should be released sometime in September and travis support should not be long.

Member

corinnekrych commented Sep 4, 2014

@abstractj I don't think the second option is doable running Swift code does require Xcode6, so we're left with the first option: disable travis for now. Xcode6 should be released sometime in September and travis support should not be long.

@danbev

This comment has been minimized.

Show comment
Hide comment
@danbev

danbev Sep 4, 2014

Member

+1 tried out the playground and it works nicely. Used Xcode6.beta6

Member

danbev commented Sep 4, 2014

+1 tried out the playground and it works nicely. Used Xcode6.beta6

@corinnekrych

This comment has been minimized.

Show comment
Hide comment
@corinnekrych

corinnekrych Sep 4, 2014

Member

@danbev Good point let me rebase and review with beta 7

Member

corinnekrych commented Sep 4, 2014

@danbev Good point let me rebase and review with beta 7

@corinnekrych

This comment has been minimized.

Show comment
Hide comment
@corinnekrych

corinnekrych Sep 4, 2014

Member

@abstractj I will disable Travis in a separate PR if you don't mind

Member

corinnekrych commented Sep 4, 2014

@abstractj I will disable Travis in a separate PR if you don't mind

@corinnekrych

This comment has been minimized.

Show comment
Hide comment
@corinnekrych

corinnekrych Sep 5, 2014

Member

@abstractj
I've created a JIRA epic on Travis build
https://issues.jboss.org/browse/AGIOS-248

@danbev rebased and tested with beta7 changes and still works good.

Ready to merge?

Member

corinnekrych commented Sep 5, 2014

@abstractj
I've created a JIRA epic on Travis build
https://issues.jboss.org/browse/AGIOS-248

@danbev rebased and tested with beta7 changes and still works good.

Ready to merge?

@danielpassos

This comment has been minimized.

Show comment
Hide comment
@danielpassos

danielpassos Sep 5, 2014

Member

👍

Tested using Xcode6-Beta7

Member

danielpassos commented Sep 5, 2014

👍

Tested using Xcode6-Beta7

@corinnekrych corinnekrych merged commit 11bf979 into aerogear:master Sep 5, 2014

1 check failed

continuous-integration/travis-ci The Travis CI build failed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment