Skip to content

Fix Swift naming convention for patch age constant#26

Draft
Copilot wants to merge 2 commits intofeat/dev-medtrumfrom
copilot/sub-pr-25
Draft

Fix Swift naming convention for patch age constant#26
Copilot wants to merge 2 commits intofeat/dev-medtrumfrom
copilot/sub-pr-25

Conversation

Copy link
Copy Markdown

Copilot AI commented Dec 25, 2025

Renamed NORMAL_PATCH_AGE to normalPatchAge to follow Swift naming conventions for constants.

Changes:

  • Updated constant declaration in PumpView.swift from uppercase to camelCase
  • Updated reference in timerColor computed property
// Before
let NORMAL_PATCH_AGE = TimeInterval.hours(80)
return abs(activatedAt.timeIntervalSinceNow) > NORMAL_PATCH_AGE ? Color.yellow : Color.loopGreen

// After
let normalPatchAge = TimeInterval.hours(80)
return abs(activatedAt.timeIntervalSinceNow) > normalPatchAge ? Color.yellow : Color.loopGreen

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Co-authored-by: Sjoerd-Bo3 <2100083+Sjoerd-Bo3@users.noreply.github.com>
Copilot AI changed the title [WIP] Add support for Medtrum Nano insulin pumps Fix Swift naming convention for patch age constant Dec 25, 2025
Copilot AI requested a review from Sjoerd-Bo3 December 25, 2025 17:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants