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

block flask version to 0.10 #256

Closed
wants to merge 4 commits into from

Conversation

boyska
Copy link
Member

@boyska boyska commented Mar 3, 2016

HEAVY TESTING: don't even consider this for merging

(I am doing tests on travis, so creating a PR is the easiest thing)

refs #254

@boyska boyska force-pushed the 254-pinpoint-flask branch 5 times, most recently from 0758473 to 45f9257 Compare March 3, 2016 19:10
@boyska boyska mentioned this pull request Mar 3, 2016
@boyska boyska force-pushed the 254-pinpoint-flask branch 7 times, most recently from 844ecaa to ff4b493 Compare March 3, 2016 19:59
@ael-code
Copy link
Member

ael-code commented Mar 4, 2016

On 03/03/2016 07:47 PM, BoySka wrote:

/HEAVY TESTING: don't even consider this for merging/

(I am doing tests on travis, so creating a PR is the easiest thing)

refs #254 #254

You can also enable Travis-ci on your branches.

@boyska
Copy link
Member Author

boyska commented Mar 4, 2016

ael:

On 03/03/2016 07:47 PM, BoySka wrote:

(I am doing tests on travis, so creating a PR is the easiest thing)

You can also enable Travis-ci on your branches.

you're right :) now I did!

@ael-code
Copy link
Member

ael-code commented Mar 5, 2016

On 03/04/2016 04:24 PM, BoySka wrote:

ael:

On 03/03/2016 07:47 PM, BoySka wrote:

(I am doing tests on travis, so creating a PR is the easiest thing)

You can also enable Travis-ci on your branches.

you're right :) now I did!

Can we close this PR now?

@boyska
Copy link
Member Author

boyska commented Mar 5, 2016

ok

@boyska boyska closed this Mar 5, 2016
@boyska boyska deleted the 254-pinpoint-flask branch July 28, 2016 08:20
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