Skip to content

Code cleanup - #111

Merged
schbetsy merged 4 commits into
mainfrom
misc-cleanup
Sep 25, 2026
Merged

Code cleanup#111
schbetsy merged 4 commits into
mainfrom
misc-cleanup

Conversation

@schbetsy

@schbetsy schbetsy commented Sep 23, 2026

Copy link
Copy Markdown
Collaborator

Changes

  • No change to behavior. Just miscellaneous code cleanup commits.

Testing

  1. Tests should pass.

Checklist

  • PR has an informative and human-readable title
  • Changes are limited to a single goal (no scope creep)
  • Code can be automatically merged (no conflicts)
  • Passes all existing automated tests
  • Any change in functionality is tested
  • New functions are documented (with a description, list of inputs, and expected output)
  • Placeholder code is flagged / future todos are captured in comments
  • Visually tested in supported browsers and devices (see checklist below 👇)
  • Project documentation has been updated
  • Reviewers requested with the Reviewers tool ➡️

@schbetsy
More efficient query for activity date range
9a5c359
@schbetsy
Break out separate parser functions
02ae81e 
for greater readability and monitoring
@schbetsy
Fix potential bugs
9535eb9 
- No reason to use datetime when we actually only save a date
- base and header segment length are identical, so we accidentally
  interchanged them
@schbetsy
Upgrade python version used in test action
84ed6eb
@schbetsy
schbetsy marked this pull request as ready for review September 25, 2026 16:15
@schbetsy
schbetsy merged commit 536442a into main Sep 25, 2026
7 checks passed
@schbetsy
schbetsy deleted the misc-cleanup branch September 25, 2026 16:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@schbetsy