Closed Shahroz16 closed 11 months ago
Pull request title looks good 👍!
If this pull request gets merged, it will not cause a new release of the software. Example: If this project's latest release version is 1.0.0
. If this pull request gets merged in, the next release of this project will be 1.0.0
. This pull request is not a breaking change.
All merged pull requests will eventually get deployed. But some types of pull requests will trigger a deployment (such as features and bug fixes) while some pull requests will wait to get deployed until a later time.
@levibostian thanks for the feedback, regarding the reutilization of the code from the npm mobile repositories. The code might be similar but it's not identical because we aren't using yarn
anywhere and relying on npm
. But in those, we are using yarn
and npm
both everywhere.
Secondly, those workflows and scripts had some extra steps which might not always be needed, for example, slack postings. Plus they consist of extra/un-utilized steps and conditions, you might also find typos like deploying cocoa pods whereas those npm modules don't deploy anything on cocoa pods.
I plan on cleaning those scripts up later on but didn't want to make that a part of this ticker to avoid scoop creep.
Great. If you could create a ticket to make node workflows/scripts more re-usable that would be great.
You can ignore all
ts
undersrc
files because it's just linting.We made sure all code-related changes were done #2 and this is just for linting and deployments