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

Removes splash patches #130

Merged
merged 1 commit into from
Dec 6, 2018
Merged

Removes splash patches #130

merged 1 commit into from
Dec 6, 2018

Conversation

jamonholmgren
Copy link
Member

This removes the splash screen patches (ref #76 and #100) because they're causing problems on bare ignite new Foo -b ir-boilerplate-bowser projects.

I've generated a new Ignite plugin that could be run on an Ignited project with ignite add splash. It's not ready yet (waiting on @leonskim and @robinheinze to review and fix whatever's wrong with it right now).

Copy link
Member

@leonskim leonskim left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@robinheinze
Copy link
Member

robinheinze commented Dec 3, 2018

@jamonholmgren This looks good to me, since it's just the removal of the patches. The CI failures don't look related...

@ugurozturk
Copy link

ugurozturk commented Dec 5, 2018

Another test confirmed #131 (comment)

@mlaco

@jamonholmgren jamonholmgren merged commit 64340d5 into master Dec 6, 2018
@jamonholmgren jamonholmgren deleted the refactor/remove-splash branch December 6, 2018 06:20
jamonholmgren added a commit that referenced this pull request Dec 6, 2018
* Call react-native link with spawn instead of run (#127)
* Removes splash patches (#130)

Note that ignite-splash will provide splash capability in the future.
Check out an early implementation at:

https://github.com/infinitered/ignite-splash
infinitered-circleci pushed a commit that referenced this pull request Dec 6, 2018
## [1.1.1](v1.1.0...v1.1.1) (2018-12-06)

### Bug Fixes

* **cli:** Call link with spawn instead of run ([#127](#127)) ([4f9e8cb](4f9e8cb))
* **cli:** Fixes hanging on install ([c6daa0a](c6daa0a)), closes [#127](#127) [#130](#130)
* **splash:** Removes splash patches ([#130](#130)) ([64340d5](64340d5))
@infinitered-circleci
Copy link

🎉 This PR is included in version 1.1.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

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.

5 participants