DATA-173 | Update Context so it can work on Mobile #32
KevinvdBurgpull-request-checks.yaml
on: pull_request
Run lint and tests
15s
Annotations
2 errors and 3 warnings
|
lib/mixpanel/tracking/WebTrackingService.test.ts > WebTrackingService > should track an event successfully:
lib/mixpanel/tracking/WebTrackingService.test.ts#L78
AssertionError: expected "spy" to be called with arguments: [ { name: 'Test Event', …(1) } ]
Received:
Number of calls: 0
❯ lib/mixpanel/tracking/WebTrackingService.test.ts:78:36
|
|
Run lint and tests
Process completed with exit code 1.
|
|
Run lint and tests
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/setup-node@v3, actions/checkout@v3, actions/cache@v3. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
|
|
Run lint and tests:
lib/mixpanel/context.test.tsx#L56
React Hook useEffect has a missing dependency: 'setEventContext'. Either include it or remove the dependency array
|
|
Run lint and tests:
lib/mixpanel/context.test.tsx#L84
React Hook useEffect has a missing dependency: 'trackPageView'. Either include it or remove the dependency array
|