-
Notifications
You must be signed in to change notification settings - Fork 2
🔖 release: [email protected], [email protected] #697
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
Conversation
Summary of ChangesHello @cruzdanilo, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! This pull request is an automated release generated by the Changesets GitHub action, bumping the Highlights
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
|
Caution Review failedThe pull request is closed. WalkthroughRemoves many Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related PRs
Suggested reviewers
🚥 Pre-merge checks | ✅ 2✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
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.
Code Review
This pull request is an automated release for version 1.1.5 of the @exactly/mobile package, generated by the Changesets action. The changes include updating the package version in package.json and generating the corresponding CHANGELOG.md entry from the consumed changeset files. The changes are correct and follow the standard release process. No issues were found.
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.
6d775c1 to
c1d01d5
Compare
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #697 +/- ##
==========================================
+ Coverage 64.54% 65.25% +0.71%
==========================================
Files 173 173
Lines 5378 5532 +154
Branches 1509 1571 +62
==========================================
+ Hits 3471 3610 +139
- Misses 1727 1737 +10
- Partials 180 185 +5
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
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.
Actionable comments posted: 1
🤖 Fix all issues with AI agents
In `@CHANGELOG.md`:
- Around line 3-11: Change the prose in the CHANGELOG.md 1.1.5 section to
lowercase and wrap lines to a maximum of 120 characters: update the "## 1.1.5"
and "### Patch Changes" list items and the three bullet entries referencing PR
`#695` and commits 56e5e47, 9bec556, and 4113494 so that the narrative text (e.g.,
"dismiss flows when finished", "fix defi screen top padding", "use expo router
ui tabs") is all lowercase and each bullet line is split/wrapped so no line
exceeds 120 characters.
3f11878 to
ebd0222
Compare
ebd0222 to
f42b164
Compare
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.
Actionable comments posted: 1
🤖 Fix all issues with AI agents
In `@server/CHANGELOG.md`:
- Around line 3-7: Update the changelog entry for the "## 0.2.53" section by
converting all narrative text to lowercase (including the heading text and the
list item) and reflowing/wrapping the long list line to a maximum of 120
characters; specifically edit the heading "## 0.2.53" and the list item that
references PR `#693` / commit `de60df2` so the sentence "Thanks [`@mainqueg`]... - ✨
validate id document photos for manteca" is entirely lowercase and broken into
wrapped lines ≤120 chars.
| ## 0.2.53 | ||
|
|
||
| ### Patch Changes | ||
|
|
||
| - [#693](https://github.com/exactly/exa/pull/693) [`de60df2`](https://github.com/exactly/exa/commit/de60df282b81ce960b3f7cb07e449dc3adfffed9) Thanks [@mainqueg](https://github.com/mainqueg)! - ✨ validate id document photos for manteca |
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.
Lowercase and wrap the 0.2.53 changelog entry.
Please switch to lowercase prose and wrap the long line to ≤120 characters.
✂️ Suggested update
## 0.2.53
-### Patch Changes
+### patch changes
-- [`#693`](https://github.com/exactly/exa/pull/693) [`de60df2`](https://github.com/exactly/exa/commit/de60df282b81ce960b3f7cb07e449dc3adfffed9) Thanks [`@mainqueg`](https://github.com/mainqueg)! - ✨ validate id document photos for manteca
+- [`#693`](https://github.com/exactly/exa/pull/693)
+ [`de60df2`](https://github.com/exactly/exa/commit/de60df282b81ce960b3f7cb07e449dc3adfffed9)
+ thanks [`@mainqueg`](https://github.com/mainqueg)!
+ ✨ validate id document photos for mantecaAs per coding guidelines: All narrative text in internal documentation prose (headings, paragraphs, list items, proper nouns, brand names) must be lowercase with no exceptions; All lines in markdown files must be no longer than 120 characters.
🤖 Prompt for AI Agents
In `@server/CHANGELOG.md` around lines 3 - 7, Update the changelog entry for the
"## 0.2.53" section by converting all narrative text to lowercase (including the
heading text and the list item) and reflowing/wrapping the long list line to a
maximum of 120 characters; specifically edit the heading "## 0.2.53" and the
list item that references PR `#693` / commit `de60df2` so the sentence "Thanks
[`@mainqueg`]... - ✨ validate id document photos for manteca" is entirely
lowercase and broken into wrapped lines ≤120 chars.
0f43676 to
919ee21
Compare
919ee21 to
d27bae2
Compare
This PR was opened by the Changesets release GitHub action. When you're ready to do a release, you can merge this and the packages will be published to npm automatically. If you're not ready to do a release yet, that's fine, whenever you add more changesets to main, this PR will be updated.
Releases
@exactly/[email protected]
Patch Changes
#696
5b80605Thanks @dieguezguille! - 💄 fix missing tamagui input styles#696
a4ab311Thanks @dieguezguille! - ⚡️ optimize asset logo component#695
56e5e47Thanks @dieguezguille! - 🐛 dismiss flows when finished#695
9bec556Thanks @dieguezguille! - 💄 fix defi screen top padding#701
6077f60Thanks @dieguezguille! - 💫 improve carousel transitions#696
f36dfa0Thanks @dieguezguille! - 🐛 fix tab screen scrolling on web#696
4099f89Thanks @dieguezguille! - 💄 improve chain logo display#696
4b22d84Thanks @dieguezguille! - 🚸 navigate successful operations to activity#695
4113494Thanks @dieguezguille! - ♻️ use expo router ui tabs#696
64c6667Thanks @dieguezguille! - 🐛 fix missing asset logos on bridgeUpdated dependencies [
de60df2,8ee587a,021a7bd]:@exactly/[email protected]
Patch Changes
#693
de60df2Thanks @mainqueg! - ✨ validate id document photos for manteca#700
8ee587aThanks @mainqueg! - 🥅 add retry to identity file upload#692
021a7bdThanks @mainqueg! - ⚰️ remove template id paramSummary by CodeRabbit
✏️ Tip: You can customize this high-level summary in your review settings.