Skip to content

Fix build, invalid JSON file in icloud component#29798

Merged
balloob merged 1 commit into
devfrom
frenck-2019-0290
Dec 9, 2019
Merged

Fix build, invalid JSON file in icloud component#29798
balloob merged 1 commit into
devfrom
frenck-2019-0290

Conversation

@frenck
Copy link
Copy Markdown
Member

@frenck frenck commented Dec 9, 2019

Description:

Our language uploads in the CI currently fail.
This is due to an invalid (JSON) translation file introduced in #28968

Related issue (if applicable): n/a

Pull request with documentation for home-assistant.io (if applicable): n/a

Example entry for configuration.yaml (if applicable): n/a

Checklist:

  • The code change is tested and works locally.
  • Local tests pass with tox. Your PR cannot be merged unless tests pass
  • There is no commented out code in this PR.
  • I have followed the development checklist

@probot-home-assistant
Copy link
Copy Markdown

Hey there @Quentame, mind taking a look at this pull request as its been labeled with a integration (icloud) you are listed as a codeowner for? Thanks!

@balloob balloob merged commit bc3b7ed into dev Dec 9, 2019
@delete-merged-branch delete-merged-branch Bot deleted the frenck-2019-0290 branch December 9, 2019 18:18
@Quentame
Copy link
Copy Markdown
Member

Quentame commented Dec 9, 2019

What ? Build failed after #28968 ?

How can it happens, since all checks passed in this previous iCloud PR 🤔 ?

@frenck
Copy link
Copy Markdown
Member Author

frenck commented Dec 9, 2019

@Quentame We don't check if JSON files are valid in our builds.
We are adding that right now 😉

@Quentame
Copy link
Copy Markdown
Member

Quentame commented Dec 9, 2019

I've just saw the PR, nice btw 😉

@lock lock Bot locked and limited conversation to collaborators Dec 10, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants