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.

Sign up for GitHub

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

Jump to bottom

Release v0.2.3 #463

Merged
merged 3 commits into from
Jan 28, 2025
Merged

Release v0.2.3 #463

merged 3 commits into from
Jan 28, 2025
+3 −3

Conversation

Sjoerd-Bo3
Copy link
Contributor

@Sjoerd-Bo3 Sjoerd-Bo3 commented Jan 26, 2025 •
edited
Loading

Update version number to 0.2.3.

Updating SHA for item below:

bjornoleh
bjornoleh previously approved these changes Jan 26, 2025
View reviewed changes
@marionbarker
Copy link
Contributor

The initial comment is missing some additions to the OmniXXX updates. Here's the complete list:

OmniBLE (DASH):

Merge pull request LoopKit/OmniBLE#140 from itsmojo/concurrent-TB-logic-fixes
Merge pull request LoopKit/OmniBLE#139 from itsmojo/improved-unacknowledged-command-recovery
Merge pull request LoopKit/OmniBLE#138 from itsmojo/alternate-DASH-getStatus
Merge pull request LoopKit/OmniBLE#137 from itsmojo/improved-PodMessageTransport-error-handling
Merge pull request LoopKit/OmniBLE#136 from itsmojo/unacknowledge-command-improvement
Merge pull request LoopKit/OmniBLE#135 from itsmojo/Omnipod-improvements
Merge pull request LoopKit/OmniBLE#134 from itsmojo/OmniBLEParser-updates

OmniKit (Eros):

Merge pull request LoopKit/OmniKit#48 from itsmojo/concurrent-TB-logic-fixes
Merge pull request LoopKit/OmniKit#47 from itsmojo/improved-unacknowledged-command-recovery
Merge pull request LoopKit/OmniKit#46 from loopandlearn/omnikit-add-clock-icon-main
Merge pull request LoopKit/OmniKit#45 from itsmojo/unacknowledge-command-improvement
Merge pull request LoopKit/OmniKit#44 from itsmojo/Omnipod-improvements

@Sjoerd-Bo3
Copy link
Contributor Author

The initial comment is missing some additions to the OmniXXX updates. Here's the complete list:

OmniBLE (DASH):

Merge pull request LoopKit/OmniBLE#140 from itsmojo/concurrent-TB-logic-fixes Merge pull request LoopKit/OmniBLE#139 from itsmojo/improved-unacknowledged-command-recovery Merge pull request LoopKit/OmniBLE#138 from itsmojo/alternate-DASH-getStatus Merge pull request LoopKit/OmniBLE#137 from itsmojo/improved-PodMessageTransport-error-handling Merge pull request LoopKit/OmniBLE#136 from itsmojo/unacknowledge-command-improvement Merge pull request LoopKit/OmniBLE#135 from itsmojo/Omnipod-improvements Merge pull request LoopKit/OmniBLE#134 from itsmojo/OmniBLEParser-updates

OmniKit (Eros):

Merge pull request LoopKit/OmniKit#48 from itsmojo/concurrent-TB-logic-fixes Merge pull request LoopKit/OmniKit#47 from itsmojo/improved-unacknowledged-command-recovery Merge pull request LoopKit/OmniKit#46 from loopandlearn/omnikit-add-clock-icon-main Merge pull request LoopKit/OmniKit#45 from itsmojo/unacknowledge-command-improvement Merge pull request LoopKit/OmniKit#44 from itsmojo/Omnipod-improvements

Fixed

marionbarker
marionbarker approved these changes Jan 26, 2025
View reviewed changes
Copy link
Contributor

@marionbarker marionbarker left a comment

Choose a reason for hiding this comment

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

Approved by code review.

MikePlante1
MikePlante1 approved these changes Jan 27, 2025
View reviewed changes
Copy link
Contributor

@MikePlante1 MikePlante1 left a comment

Choose a reason for hiding this comment

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

✅ Built to a simulator to confirm no linting.
✅ Simple code review, as there are no real changes to code, just version number and SHA update.

@Sjoerd-Bo3 Sjoerd-Bo3 merged commit 7f755b9 into dev Jan 28, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@polscm32 polscm32 polscm32 approved these changes

@MikePlante1 MikePlante1 MikePlante1 approved these changes

@bjornoleh bjornoleh bjornoleh approved these changes

@dnzxy dnzxy dnzxy approved these changes

@marionbarker marionbarker marionbarker approved these changes

@aug0211 aug0211 Awaiting requested review from aug0211 aug0211 is a code owner

@AndreasStokholm AndreasStokholm Awaiting requested review from AndreasStokholm AndreasStokholm is a code owner

@t1dude t1dude Awaiting requested review from t1dude t1dude is a code owner

@avouspierre avouspierre Awaiting requested review from avouspierre

@tmhastings tmhastings Awaiting requested review from tmhastings

@dsnallfot dsnallfot Awaiting requested review from dsnallfot

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@Sjoerd-Bo3 @marionbarker @dnzxy @bjornoleh @MikePlante1 @polscm32