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

github: suggestion: reduce jitter #193

Open
kamilsk opened this issue Jan 9, 2024 · 0 comments
Open

github: suggestion: reduce jitter #193

kamilsk opened this issue Jan 9, 2024 · 0 comments
Assignees
@kamilsk
Labels
effort: easy An issue is easy to implement. impact: high An issue has high impact. scope: code An issue related to source code. type: improvement A new improvement proposal.
Milestone
GCC, v0.1

Comments

@kamilsk
Copy link
Member

kamilsk commented Jan 9, 2024

Motivation: it limits available capacity for the day.

Details

maintainer/internal/command/github/contribution/suggest.go

Line 50 in 020d048

suggestion.Time = suggestion.Time.Add(jitter.FullRandom().Apply(time.Hour))

To do

I must change it to a "target-specific" value but limit it to top.
@kamilsk kamilsk added scope: code An issue related to source code. type: improvement A new improvement proposal. impact: high An issue has high impact. effort: easy An issue is easy to implement. labels Jan 9, 2024
@kamilsk kamilsk added this to the GCC, v0.1 milestone Jan 9, 2024
@kamilsk kamilsk self-assigned this Jan 9, 2024
@github-project-automation github-project-automation bot moved this to Backlog in 🏗 Platform Jan 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@kamilsk kamilsk

Labels
effort: easy An issue is easy to implement. impact: high An issue has high impact. scope: code An issue related to source code. type: improvement A new improvement proposal.
Projects
🏗 Platform
Status: Backlog
Milestone
GCC, v0.1
Development

No branches or pull requests
1 participant
@kamilsk