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

New Library Request: expo-navigation-bar #51957

Closed
kirillzyusko opened this issue Nov 4, 2024 · 6 comments
Closed

New Library Request: expo-navigation-bar #51957

kirillzyusko opened this issue Nov 4, 2024 · 6 comments
Assignees
Labels
AutoAssignerAppLibraryReview Auto assign someone to review a new library being added to App Weekly KSv2

Comments

@kirillzyusko
Copy link
Contributor

kirillzyusko commented Nov 4, 2024

In order to properly evaluate if a new library can be added to package.json, please fill out this request form. It will be automatically assigned someone from our review team that will go through and vet the library.

In order to add any new production dependency, it must be approved by the App Deployer team. They will evaluate the library and decide if it's something we want to move forward with or if other alternatives should be explored.

Note: This is only for production dependencies. While we don't want people to add packages to dev-dependencies willy-nilly, we recognize that there isn't as great of a need there to secure them.

Name of library:

Details

@kirillzyusko kirillzyusko added AutoAssignerAppLibraryReview Auto assign someone to review a new library being added to App Weekly KSv2 labels Nov 4, 2024
Copy link

melvin-bot bot commented Nov 4, 2024

Triggered auto assignment to @jasperhuangg (AutoAssignerAppLibraryReview), see https://stackoverflowteams.com/c/expensify/questions/17737 for more details.

Copy link

melvin-bot bot commented Nov 4, 2024

New Library Review

  • Are all the answers in the main description filled out properly and make sense?
  • Who maintains the library and how well is it maintained?
  • How viable are the alternatives?
  • Should we build it ourselves instead?

Once these questions are answered, start a thread in #engineering-chat, ping the @app_deployers group, and call for a vote to accept the new library. Once the vote is complete, update this issue with the outcome and procede accordingly. Here is a sample post:

Hey @app_deployers,

There is a request to add a new library to App that we need to consider. Please look at this GH and then vote :+1: or :-1: on accepting this new library or not.

GH_LINK

@jasperhuangg
Copy link
Contributor

jasperhuangg commented Nov 6, 2024

Are all the answers in the main description filled out properly and make sense?

Yes

Who maintains the library and how well is it maintained?

Expo, seems to be very well maintained.

How viable are the alternatives?

There is another library, react-native-navigation-bar-color, but its popularity seems to be dying down. The package may not support Fabric/bridgeless mode, etc., so it's not ideal for us to use.

Should we build it ourselves instead?

Probably not, Expo is a pretty reputable company that releases some very performant and well-maintained. It seems easier for us to just leverage this library.

@jasperhuangg
Copy link
Contributor

@jasperhuangg
Copy link
Contributor

Latest update here

@jasperhuangg
Copy link
Contributor

We're not moving forward with this library request since we have longer-term solutions that we're working on that will fix this issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
AutoAssignerAppLibraryReview Auto assign someone to review a new library being added to App Weekly KSv2
Projects
None yet
Development

No branches or pull requests

2 participants