Skip to content
This repository was archived by the owner on May 8, 2023. It is now read-only.

ci: Add "true" support for multi-line command #24

Merged
merged 2 commits into from
Nov 3, 2016

Conversation

jcfr
Copy link
Contributor

@jcfr jcfr commented Nov 3, 2016

It is now possible to specify python command as depicted in the
example below.

Note that the scikit-ci-schema still need to been updated to validate
against python command. See scikit-build/scikit-ci-schema#1

test:
  commands:
    - python: print("single_line")
    - python: "for letter in ['a', 'b', 'c']: print(letter)"
    - python: |
              for index in range(3):
                  with open("file_%s" % index, "w") as output:
                      output.write("")

@jcfr jcfr force-pushed the support-python-commands branch from f970676 to 89401a8 Compare November 3, 2016 15:31
@codecov-io
Copy link

codecov-io commented Nov 3, 2016

Current coverage is 96.68% (diff: 100%)

Merging #24 into master will increase coverage by 0.27%

@@             master        #24   diff @@
==========================================
  Files             5          5          
  Lines           279        302    +23   
  Methods           0          0          
  Messages          0          0          
  Branches         42         44     +2   
==========================================
+ Hits            269        292    +23   
  Misses            4          4          
  Partials          6          6          

Powered by Codecov. Last update 66746e3...7c45174

@jcfr jcfr force-pushed the support-python-commands branch from 89401a8 to 7c45174 Compare November 3, 2016 15:32
jcfr added 2 commits November 3, 2016 11:57
It is now possible to specify python command as depicted in the
example below.

Note that the scikit-ci-schema still need to been updated to validate
against python command. See scikit-build/scikit-ci-schema#1

```yml
test:
  commands:
    - python: print("single_line")
    - python: "for letter in ['a', 'b', 'c']: print(letter)"
    - python: |
              for index in range(3):
                  with open("file_%s" % index, "w") as output:
                      output.write("")
```
Waiting the following PR is integrated upstream, this commit implements
a workaround: https://bitbucket.org/ruamel/yaml/pull-requests/13
@jcfr jcfr force-pushed the support-python-commands branch from 7c45174 to 30fa31b Compare November 3, 2016 15:58
@jcfr
Copy link
Contributor Author

jcfr commented Nov 3, 2016

Travis OSX seems to have a back log. Consider the build passes on Windows (Python 2.x, 3.x) and Linux (Python 2.x, 3.x), I am integrating.

@jcfr jcfr merged commit 81cfe74 into master Nov 3, 2016
@jcfr jcfr deleted the support-python-commands branch November 3, 2016 16:05
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants