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

Update the validator & fix Fox model#281

Merged
emackey merged 3 commits intomasterfrom
upgrade-ci
Dec 5, 2020
Merged

Update the validator & fix Fox model#281
emackey merged 3 commits intomasterfrom
upgrade-ci

Conversation

@lexaknyazev
Copy link
Member

No description provided.

@emackey
Copy link
Member

emackey commented Dec 4, 2020

Does Fox.glb need fixing too? I guess the CI will tell us...

@emackey
Copy link
Member

emackey commented Dec 4, 2020

Oh it is fixed, it just wasn't obvious from the way GitHub shows diffs.

@emackey
Copy link
Member

emackey commented Dec 4, 2020

This all looks good, except Travis is broken/hung/slow throughout most of the Khronos repos, so we don't have a CI result.

@donmccurdy
Copy link
Contributor

/cc #282

@emackey
Copy link
Member

emackey commented Dec 5, 2020

I merged @donmccurdy's #282 in here, so we could see the new CI run. It appears to have run twice for the same commit, once for "push" and once for "pull_request". The latter finished in 54 seconds, but the "push" one is still trying to clone the repo, several minutes later. Is that a fluke? Should we only allow one build?

@emackey
Copy link
Member

emackey commented Dec 5, 2020

Well it passed, after 18 minutes. We can fine-tune in a follow-up PR. Merging.

@emackey emackey merged commit 0137bc5 into master Dec 5, 2020
@emackey emackey deleted the upgrade-ci branch December 5, 2020 15:08
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.

3 participants