Skip to content

Conversation

@devin-ai-integration
Copy link

Fix tech debt around duration config

Resolves #35

This PR addresses the tech debt around duration configuration by:

  1. Creating a new duration.Duration type to replace integer seconds in config
  2. Using util.TimeRange consistently where appropriate
  3. Updating code to use the new duration types

Changes:

  • Added a new pkg/util/duration package with Duration and RandomDuration types
  • Replaced integer seconds fields in config structs with duration.Duration types
  • Updated validation functions for all new duration fields
  • Updated code to use the new duration types with .ToDuration() method
  • Ensured proper use of util.TimeRange for randomized durations

Link to Devin run: https://app.devin.ai/sessions/c7aa6d5f0dc844ceb6f99ff69c62cfa7
User: [email protected]

@devin-ai-integration
Copy link
Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

@olegbbtr olegbbtr self-assigned this Apr 30, 2025
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.

Tech debt around duration config

2 participants