-
Notifications
You must be signed in to change notification settings - Fork 2
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
Bump react-native-haptic-feedback from 2.2.0 to 2.3.3 in /VAMobile #9873
Bump react-native-haptic-feedback from 2.2.0 to 2.3.3 in /VAMobile #9873
Conversation
Bumps [react-native-haptic-feedback](https://github.com/mkuczera/react-native-haptic-feedback) from 2.2.0 to 2.3.3. - [Release notes](https://github.com/mkuczera/react-native-haptic-feedback/releases) - [Commits](mkuczera/react-native-haptic-feedback@v2.2.0...v2.3.3) --- updated-dependencies: - dependency-name: react-native-haptic-feedback dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]>
@theodur I updated the pods and I am still getting the error if you can take a look |
@Sparowhawk The issue was the package needed to be mocked in our unit test setup. They must have changed something with how the package is referenced in the latest release. I'm running Detox tests to make sure everything passes before approving and merging. Edit: This is one of those packages that, even though should work with a minor update, should still be tested physically, since there's no way to be certain that haptics still works from the tests. I kicked on an on-demand build. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Detox tests passed, and I confirmed haptics is still working on a local build
Bumps react-native-haptic-feedback from 2.2.0 to 2.3.3.
Release notes
Sourced from react-native-haptic-feedback's releases.
Commits
8cdf0f3
remove yarn.lock7668b09
adjust example project usage5bb92f6
Merge pull request #139 from huextrat/fix/missing-types-export6c5b540
fix: export types from index3254566
version bumpd1516d5
Merge pull request #135 from huextrat/fix/types65c8b10
Merge pull request #136 from huextrat/build/bump-compile-target-sdkae7a3f8
build(android): bump compile & target SDK to 3416508f2
fix: types55b7150
Merge pull request #133 from WoLewicki/@wolewicki/fix-newarch
Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting
@dependabot rebase
.Dependabot commands and options
You can trigger Dependabot actions by commenting on this PR:
@dependabot rebase
will rebase this PR@dependabot recreate
will recreate this PR, overwriting any edits that have been made to it@dependabot merge
will merge this PR after your CI passes on it@dependabot squash and merge
will squash and merge this PR after your CI passes on it@dependabot cancel merge
will cancel a previously requested merge and block automerging@dependabot reopen
will reopen this PR if it is closed@dependabot close
will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually@dependabot show <dependency name> ignore conditions
will show all of the ignore conditions of the specified dependency@dependabot ignore this major version
will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this minor version
will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this dependency
will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)