Skip to content

Remove gutenberg-mobile submodule while issues are sorted out#11301

Closed
danielbachhuber wants to merge 2 commits intomasterfrom
rm-mobile-submodule
Closed

Remove gutenberg-mobile submodule while issues are sorted out#11301
danielbachhuber wants to merge 2 commits intomasterfrom
rm-mobile-submodule

Conversation

@danielbachhuber
Copy link
Copy Markdown
Member

@danielbachhuber danielbachhuber commented Oct 31, 2018

@youknowriad
Copy link
Copy Markdown
Contributor

The issues are not happening consistently. I'd be fine with removing but I prefer if we give the mobile team some time to figure it out and see if it can be improved.

@danielbachhuber
Copy link
Copy Markdown
Member Author

prefer if we give the mobile team some time to figure it out and see if it can be improved.

Can we please remove for now, and then restore when it's improved? As it stands, it's blocking the development workflow for multiple people.

@danielbachhuber
Copy link
Copy Markdown
Member Author

I sorted my more immediate problem by including npm run mobile-submodule-update in my build script.

@danielbachhuber danielbachhuber deleted the rm-mobile-submodule branch October 31, 2018 14:37
@earnjam
Copy link
Copy Markdown
Contributor

earnjam commented Oct 31, 2018

Gutenberg includes gutenberg-mobile, which includes gutenberg, which includes gutenberg-mobile, which includes gutenberg

image

Seems like there has to be a better way

@youknowriad
Copy link
Copy Markdown
Contributor

I know it's not great but my point is that we should try to be inclusive with the mobile team. Including their tests mean we don't break mobile inadvertently. We're all on the same boat, we should try to merge the repositories I think.

That doesn't mean we can't do quick wins for now.

@youknowriad
Copy link
Copy Markdown
Contributor

I personally prefer this approach for now #11304

@danielbachhuber
Copy link
Copy Markdown
Member Author

I personally prefer this approach for now

Yep, seems fine.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants