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

maintenance #113

Merged
merged 6 commits into from
Mar 28, 2024
Merged

maintenance #113

merged 6 commits into from
Mar 28, 2024

Conversation

necipallef
Copy link
Contributor

  • disabled Flipper according to this comment.
  • overall improvements on docs
  • refactored tests

Copy link

github-actions bot commented Mar 27, 2024

Coverage report

St.
Category Percentage Covered / Total
🔴 Statements 42.4% 53/125
🔴 Branches 22.22% 8/36
🔴 Functions 42.5% 17/40
🔴 Lines 40.5% 49/121

Test suite run success

10 tests passing in 3 suites.

Report generated by 🧪jest coverage report action from a61150f

Show full coverage report
St File % Stmts % Branch % Funcs % Lines Uncovered Line #s
🔴 All files 42.4 22.22 42.5 40.49
🟡  FingerprintJsProAgent.ts 50 40 66.66 50 22-39,69
🟡  FingerprintJsProContext.ts 75 100 0 75 5
🟢  FingerprintJsProProvider.tsx 93.33 50 100 93.33 43
🔴  errors.ts 16 11.11 16 16 8-33,56-249,272-273,319-323,330-364,368
🔴  index.ts 0 0 0 0
🔴  types.ts 0 0 0 0
🟢  useVisitorData.ts 100 100 100 100

@necipallef
Copy link
Contributor Author

necipallef commented Mar 27, 2024

I've just updated Rollup. After running yarn build with the old and new config, the diff are minimal, I've attached screenshots of them. Also, no diff exists for fpjs-pro-react-native.esm.js file.

diff1: https://www.diffchecker.com/PFkKkE8a/
diff2: https://www.diffchecker.com/YwledIDX/
diff3: https://www.diffchecker.com/MshutveO/

fpjs-pro-react-native d ts diff
fpjs-pro-react-native cjs js diff

package.json Outdated Show resolved Hide resolved
Copy link

This PR will not create a new release 🚀

@ilfa ilfa requested review from Alexey-Verkhovsky and removed request for Alexey-Verkhovsky March 28, 2024 14:15
@necipallef necipallef merged commit 474aafc into test Mar 28, 2024
6 checks passed
@necipallef necipallef deleted the chore/maintenance branch March 28, 2024 14:41
@necipallef necipallef restored the chore/maintenance branch March 28, 2024 14:42
@fingerprint-dx-team
Copy link

🎉 This PR is included in version 3.0.2-test.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

@fingerprint-dx-team
Copy link

🎉 This PR is included in version 3.0.2 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants