Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow using single core repository setup #510

Merged
merged 10 commits into from
Apr 18, 2023
Merged

Conversation

Walingar
Copy link
Collaborator

Proposed Changes

  • Move buildscripts from compose-multiplatform repo to core
  • Introduce DEVELOPMENT.md that is mostly copied from compose-multiplatform repo
  • Some changes in scripts around to make single repository setup working

Testing

Test: run tasks from newely created :mpp:* gradle project

Issues Fixed

No issues related to the PR

@eymar
Copy link
Member

eymar commented Apr 18, 2023

So to test it, I can simply clone the compose-multiplaform-core repo (nr/core-standalone branch) without compose-multiplatform and submodules. And it should work, right?

@Walingar
Copy link
Collaborator Author

@eymar yes! Except android everything should work fine

DEVELOPMENT.md Outdated Show resolved Hide resolved
Copy link

@dima-avdeev-jb dima-avdeev-jb left a comment

Choose a reason for hiding this comment

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

Checked. It works with iOS samples.

@dima-avdeev-jb
Copy link

Also, I have a not important suggestion to rename DEVELOPMENT.md to MULTIPLATFORM.md

@Walingar Walingar merged commit d4b2366 into jb-main Apr 18, 2023
@Walingar Walingar deleted the nr/core-standalone branch April 18, 2023 17:04
eymar pushed a commit that referenced this pull request May 1, 2023
* Move README.md from compose-multiplatform repo to core

* Move scripts from compose-multiplatform to core

* Move fonts from compose multiplatform repo to core

* Update scripts to support single core repository setup

---------

Co-authored-by: dima.avdeev <99798741+dima-avdeev-jb@users.noreply.github.com>
eymar pushed a commit that referenced this pull request May 1, 2023
* Move README.md from compose-multiplatform repo to core

* Move scripts from compose-multiplatform to core

* Move fonts from compose multiplatform repo to core

* Update scripts to support single core repository setup

---------

Co-authored-by: dima.avdeev <99798741+dima-avdeev-jb@users.noreply.github.com>
igordmn pushed a commit that referenced this pull request Jun 12, 2023
* Move README.md from compose-multiplatform repo to core

* Move scripts from compose-multiplatform to core

* Move fonts from compose multiplatform repo to core

* Update scripts to support single core repository setup

---------

Co-authored-by: dima.avdeev <99798741+dima-avdeev-jb@users.noreply.github.com>
igordmn pushed a commit that referenced this pull request Jun 22, 2023
* Move README.md from compose-multiplatform repo to core

* Move scripts from compose-multiplatform to core

* Move fonts from compose multiplatform repo to core

* Update scripts to support single core repository setup

---------

Co-authored-by: dima.avdeev <99798741+dima-avdeev-jb@users.noreply.github.com>
igordmn pushed a commit that referenced this pull request Jun 22, 2023
* Move README.md from compose-multiplatform repo to core

* Move scripts from compose-multiplatform to core

* Move fonts from compose multiplatform repo to core

* Update scripts to support single core repository setup

---------

Co-authored-by: dima.avdeev <99798741+dima-avdeev-jb@users.noreply.github.com>
igordmn pushed a commit that referenced this pull request Jan 30, 2024
This also works around sending an empty test suite to FTL from savedstate-ktx that is currently causing failures.

Test: cd navigation && ./gradlew bOS
Change-Id: I94764a195b488210fd5af5e11079900ae4cc2a47

This is an imported pull request from androidx#510.

Resolves #510
Github-Pr-Head-Sha: 0651b25
GitOrigin-RevId: 8d7dc9c
Change-Id: I14d1e8f6132daba6fde82995a1bfb9f01dd30281
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