Skip to content

Conversation

jpnurmi
Copy link
Member

@jpnurmi jpnurmi commented Oct 3, 2020

No description provided.

Everything matching .gitignore (git ls-files -i --exclude-standard).
First of all, connectivity_macos doesn't need its own example, but the
shared example in the main connectivity_plus package will do. Secondly,
the rest of connectivity_macos was moved to connectivity_plus_macos.
Probably because "src" was listed in .gitignore.
@jpnurmi jpnurmi requested a review from slightfoot October 3, 2020 11:28
@jpnurmi jpnurmi force-pushed the feature/cleanup-n-fixup branch from 76c1416 to 09ba95e Compare October 3, 2020 13:54
Apparently it was a symbolic link before, but it got screwed up in the
transition. The macOS version is now in a separate repo, so there's no
reason to have a shared folder for Darwin in the main connectivity pkg.
@jpnurmi jpnurmi force-pushed the feature/cleanup-n-fixup branch from d75a9f4 to 3711855 Compare October 3, 2020 14:10
@jpnurmi
Copy link
Member Author

jpnurmi commented Oct 3, 2020

After these fixes, all examples run on all platforms that currently have the respective plugin in main. I have tested each example on Android, iOS, macOS, and Linux, where applicable. /cc @mhadaily

@mhadaily
Copy link
Member

mhadaily commented Oct 7, 2020

LGTM @jpnurmi GREAT JOB.

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.

2 participants