Skip to content

chore(clerk-expo-passkeys): fix breaking changes - #4601

Merged
AlexNti merged 5 commits into
mainfrom
expo-passkeys-fix-after-pnpm
Nov 20, 2024
Merged

chore(clerk-expo-passkeys): fix breaking changes#4601
AlexNti merged 5 commits into
mainfrom
expo-passkeys-fix-after-pnpm

Conversation

@AlexNti

@AlexNti AlexNti commented Nov 19, 2024

Copy link
Copy Markdown
Contributor

Description

In this commit, we are importing the Buffer from buffer instead of node:buffer.

We are also moving the @clerk/expo-passkeys back to devDependencies at @clerk/clerk-expo

Checklist

  • pnpm test runs as expected.
  • pnpm build runs as expected.
  • (If applicable) JSDoc comments have been added or updated for any package exports
  • (If applicable) Documentation has been updated

Type of change

  • 🐛 Bug fix
  • 🌟 New feature
  • 🔨 Breaking change
  • 📖 Refactoring / dependency upgrade / documentation
  • other:

@changeset-bot

changeset-bot Bot commented Nov 19, 2024

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: bb7267f

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 2 packages
Name Type
@clerk/expo-passkeys Patch
@clerk/clerk-expo Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@AlexNti
AlexNti marked this pull request as ready for review November 19, 2024 13:55
Comment thread packages/expo/package.json Outdated
Comment thread packages/expo/package.json
Comment thread packages/expo-passkeys/src/utils.ts
@AlexNti
AlexNti requested a review from jacekradko November 19, 2024 17:07
@AlexNti
AlexNti force-pushed the expo-passkeys-fix-after-pnpm branch from 3b3b808 to 8193da1 Compare November 19, 2024 17:10
settings: {
'import/ignore': ['node_modules/react-native/index\\.js$'],
},
rules: {

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🎉

@jacekradko jacekradko left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:shipit:

@AlexNti
AlexNti merged commit 725b885 into main Nov 20, 2024
@AlexNti
AlexNti deleted the expo-passkeys-fix-after-pnpm branch November 20, 2024 11:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants