Feature/e2e #417
Feature/e2e #417
Conversation
tvanlaerhoven
force-pushed
the
feature/e2e_cavy
branch
2 times, most recently
from
84e92fe to
5900554
Compare
tvanlaerhoven
force-pushed
the
feature/e2e_cavy
branch
from
898f00e to
4dfc6bf
Compare
tvanlaerhoven
force-pushed
the
feature/e2e_cavy
branch
from
4a5a10b to
72ef8b4
Compare
| run: | | ||
| pod install | ||
| pod update |
There was a problem hiding this comment.
Is deze nodig?
Een pod install zorgt ervoor dat de pods die in de repo onder de podfile.lock zitten worden geinstalleerd. Door een pod update te doen ga je de locks negeren en hogere pod versies installeren. De uiteindelijk build is dan niet tegen dezelfde pods als in de repo.
Ik zou verwachten dat als er een pod update moet gebeuren, die dan door ons gedaan wordt en gecommit wordt in de repo. Deze e2e moet gewoon oppikken wat daarin beslist werd?
There was a problem hiding this comment.
Hoe zorgen we er dan best voor dat we altijd tegen de laatste versie van alle native dependencies builden?
|
|
||
| - (BOOL)application:(UIApplication *)application didFinishLaunchingWithOptions:(NSDictionary *)launchOptions | ||
| { | ||
| RCTBridge *bridge = [[RCTBridge alloc] initWithDelegate:self launchOptions:launchOptions]; |
There was a problem hiding this comment.
Deze setup is ok als start, maar zal nog een extra pass nodig hebben om ook de new architecture te ondersteunen. Maar laten we dit nu eerst vooral al opstarten onder deze vorm.
Added some e2e tests to github workflows.