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

feat: remove rnx-start command #139

Merged
merged 1 commit into from
Apr 16, 2021
Merged

feat: remove rnx-start command #139

merged 1 commit into from
Apr 16, 2021

Conversation

tido64
Copy link
Member

@tido64 tido64 commented Apr 16, 2021

This command currently does not provide any additional value over react-native start.

Resolves #101.

Follow-up: #140

This command currently does not provide any additional value over
`react-native start`.
tido64 added a commit to tido64/fluentui-react-native that referenced this pull request Apr 16, 2021
`rnx-start` is being removed because it currently does not provide any
value over vanilla `react-native start`.

See microsoft/rnx-kit#139
@tido64 tido64 merged commit 0358ff6 into main Apr 16, 2021
@tido64 tido64 deleted the tido/remove-rnx-start branch April 16, 2021 23:25
tido64 added a commit to microsoft/fluentui-react-native that referenced this pull request Apr 16, 2021
`rnx-start` is being removed because it currently does not provide any
value over vanilla `react-native start`.

See microsoft/rnx-kit#139
chiuam pushed a commit to chiuam/fluentui-react-native that referenced this pull request Apr 28, 2021
`rnx-start` is being removed because it currently does not provide any
value over vanilla `react-native start`.

See microsoft/rnx-kit#139
HeyImChris pushed a commit to microsoft/fluentui-react-native that referenced this pull request May 11, 2021
* prototype

* style gradient

* componentize shimmer

* prototype

* style gradient

* Android theme (#613)

* initial android theme

* mapped a lot of colors, updated readme, added to fluent-tester-app

* Change files

* removed unneccessary dependencies

* added android typography

* updated versioning, removed apple specific changes

* making android-theme default for the platform

* update fluent-tester dependency version...

* resolving comments...

* resolving more comments...

Co-authored-by: Tushar Masane <tushar@AZH-NT-02.middleeast.corp.microsoft.com>

* 📦 applying package updates ***NO_CI***

* Add repository link for @fluentui/react-native (#649)

* Change files

* Remove change file

* Add repository link

* Change files

* Add a nil check to Avatar image property (#650)

* Add a nil check to AvatarData image

* Change files

* Update Radio Button focus border style to be WCAG 2.1 compliant (#653)

* Change files

* Remove change file

* Update radio button focus border style

* Change files

* Add the rainbow gradient border image test case to experimental avatar (#654)

* Add the rainbow gradient test case

* Change files

* Add apple theme static libraries to NuGet package (#655)

* Add apple theme static libraries to NuGet, add NuGet Publish PR job

* Fix path in nuspec

* Comment out NugetPublish PR Job

* Fix npm publish pipeline (hopefully) (#656)

* fix typos in android-theme

* Change files

* 📦 applying package updates ***NO_CI***

* Android Svg Support  (#652)

* added svg, transformer to android

* base64 svgs do not load, font files do not link

* Change files

* bump package versions manually

* Change files

* upgrade react-native-svg

* Change files

* revert 'bump package versions manually'

* Change files

* 📦 applying package updates ***NO_CI***

* Fix typo in nuspec (#659)

* Fix typo in nuspec

* Comment out NuGet Publish again

* Add Icon support for macOS (#645)

* Add macOS support

* Add link to issue regarding direct SVG imports

* Replace react-native-svg with @microsoft/react-native-svg-desktop

* Use react-native-svg now that it supports macOS

* Upgrade react-native-svg to 12.1.1-0 everywhere

* Update macos podfile

* Change files

* 📦 applying package updates ***NO_CI***

* chore: `rnx-start` is deprecated (#660)

`rnx-start` is being removed because it currently does not provide any
value over vanilla `react-native start`.

See microsoft/rnx-kit#139

* Fix more typos in the nuspec (#661)

* Fix typos, uncomment pipeline

* comment out pipeline again

* Revamp Android ThemePicker (#664)

* updated themepicker for android

* Change files

* add correct picker...

* ignore rn-picker depcheck...

* separated android themepicker...

* added dropdown styles...

* add types for dropdown..

* Add a Contributing markdown file (#555)

* Initial Commit

* Restructuring

* Rewrite Slots summary

* Update Guide based on feedback

* More fixes

* Update from PR feedback

* Add RNSVG dependency to Icon and update RNSVG version throughout (#668)

* Add button icon tests specific to win32

* Change files

* Change files

* Add dependency to rnsvg

* Add RNSVG dependency to ios and mac

* Remove rnsvg dep from ios and macos due to failed depcheck

* Add rnsvg to ios and macos againt but also add to ignores for depcheck

* Add explicit react-native-svg dependency to FURN Icon (#666)

* Add direct dependency on RNSVG in FURN Icon

* Change files

* Update yarn.lock with react-native-svg 12.1.1

* componentize shimmer

* move shimmer to experimental

* create slots for shapes

* revamp shimmer slots

* implement shimmerElement to group sub-elements

* Change files

* add shimmer to android and accessibility support

* use useRef hook and union types

* remove rect & circle slots

* documentation

Co-authored-by: Tushar Anil Masane <tamasane@gmail.com>
Co-authored-by: Tushar Masane <tushar@AZH-NT-02.middleeast.corp.microsoft.com>
Co-authored-by: UI-Fabric-RN-Bot <uifrnbot@microsoft.com>
Co-authored-by: Krystal Siler <krsiler@microsoft.com>
Co-authored-by: Saad Najmi <saadnajmi2@gmail.com>
Co-authored-by: Adam Gleitman <adam.gleitman@gmail.com>
Co-authored-by: Tommy Nguyen <4123478+tido64@users.noreply.github.com>
Co-authored-by: warren-ms <63264034+warren-ms@users.noreply.github.com>
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.

rnx-start does not allow specifying custom config files
2 participants