Skip to content
This repository has been archived by the owner on Jan 10, 2023. It is now read-only.

Handle newlines in .bowerrc #3

Merged
merged 1 commit into from
May 17, 2015
Merged

Handle newlines in .bowerrc #3

merged 1 commit into from
May 17, 2015

Conversation

mgeisler
Copy link
Contributor

When setting $bower_components, the .bowerrc file was first read into
an environment variable, an this variable was then embedded in a
$(python -c "...") construct. This failed for my build due to newlines
in the .bowerrc file:

$ bash <(curl -s https://codecov.io/bash)
  File "<string>", line 1
    import json as j; print j.loads('{
                                     ^
SyntaxError: EOL while scanning string literal

We now simply let the Python JSON module read the .bowerrc file.

When setting $bower_components, the .bowerrc file was first read into
an environment variable, an this variable was then embedded in a
$(python -c "...") construct. This failed for my build due to newlines
in the .bowerrc file:

  $ bash <(curl -s https://codecov.io/bash)
    File "<string>", line 1
      import json as j; print j.loads('{
                                       ^

  SyntaxError: EOL while scanning string literal

We now simply let the Python JSON module read the .bowerrc file.
stevepeak added a commit that referenced this pull request May 17, 2015
@stevepeak stevepeak merged commit bcb37d9 into codecov:master May 17, 2015
@stevepeak
Copy link
Contributor

Great job! Thank you for the contribution!

@mgeisler mgeisler mentioned this pull request May 18, 2015
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.

None yet

2 participants