Skip to content

Gdr 771#72

Merged
bczech merged 2 commits intomainfrom
GDR-771
Aug 21, 2025
Merged

Gdr 771#72
bczech merged 2 commits intomainfrom
GDR-771

Conversation

@bczech
Copy link
Contributor

@bczech bczech commented Aug 12, 2025

Description

What changed?

Related JIRA issue: GDR-771

Why was it changed?

To fix usage of ifelse

Checklist for sustainable code base

  • I added tests for any code changed/added
  • I added documentation for any code changed/added
  • I made sure naming of any new functions is self-explanatory and consistent

Logistic checklist

  • Package version bumped
  • Changelog updated

Screenshots (optional)

@bczech bczech requested a review from a team as a code owner August 12, 2025 06:15
@bczech bczech requested review from darsoo and gladkia and removed request for a team August 12, 2025 06:15
Copy link
Contributor

@j-smola j-smola left a comment

Choose a reason for hiding this comment

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

The add_day0_data function is exported but it does not have tests 😢

@bczech bczech merged commit 8e56860 into main Aug 21, 2025
3 of 4 checks passed
@bczech bczech deleted the GDR-771 branch August 21, 2025 07:23
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.

3 participants

Comments