Commit 043e298
authored
chore(deps): upgrade all dependencies including go-cron v0.7.0 (mcuadros#404)
## Summary
- Upgrade go-cron from v0.6.0 to v0.7.0 and all other project
dependencies to latest versions
- Refactor schedule validation to use go-cron v0.7.0's new
`ValidateSpec` API for cleaner code
- Fix test race condition for go-cron v0.7.0 compatibility
## Changes
### Dependency Upgrades
**Direct dependencies:**
- `github.com/netresearch/go-cron`: v0.6.0 → v0.7.0
- `github.com/docker/cli`: v29.1.2 → v29.1.3
- `golang.org/x/crypto`: v0.45.0 → v0.46.0
- `golang.org/x/term`: v0.37.0 → v0.38.0
- `golang.org/x/text`: v0.31.0 → v0.32.0
**Indirect dependencies:**
- `go.opentelemetry.io/contrib/instrumentation/net/http/otelhttp`:
v0.63.0 → v0.64.0
- `go.opentelemetry.io/otel`: v1.38.0 → v1.39.0
- `go.opentelemetry.io/otel/metric`: v1.38.0 → v1.39.0
- `go.opentelemetry.io/otel/trace`: v1.38.0 → v1.39.0
- `golang.org/x/sys`: v0.38.0 → v0.39.0
### Code Improvements
**cli/doctor.go:**
- Refactored `validateCronSchedule` to use go-cron v0.7.0's new
`ValidateSpec` API
- Reduced 35 lines of manual validation code to a single function call
- Maintains full backward compatibility with all cron formats
**core/scheduler_edge_cases_test.go:**
- Fixed race condition in `TestSchedulerConcurrentOperations`
- Stop scheduler before adding jobs to avoid go-cron's internal map race
### New go-cron v0.7.0 Features Available
The upgrade enables future use of these new capabilities:
- **Schedule introspection**: `Bounds()`, `Fields()`, `Matches()`,
`NextN()`, `Between()`
- **Backward time traversal**: `Prev()` method for missed execution
detection
- **Run-once jobs**: `WithRunOnce()`, `AddOnceFunc()` for
single-execution tasks
- **Jitter wrappers**: Prevent thundering herd in distributed systems
- **Jenkins H hash expressions**: Deterministic load distribution
- **Year field support**: Multi-year scheduling capability
## Test plan
- [x] All unit tests pass
- [x] Race detector passes (`go test -race ./...`)
- [x] Lint checks pass
- [x] Build succeedsFile tree
5 files changed
+58
-84
lines changed- cli
- core
5 files changed
+58
-84
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
303 | 303 | | |
304 | 304 | | |
305 | 305 | | |
306 | | - | |
307 | 306 | | |
308 | 307 | | |
309 | 308 | | |
310 | 309 | | |
311 | | - | |
| 310 | + | |
312 | 311 | | |
313 | 312 | | |
314 | 313 | | |
| |||
326 | 325 | | |
327 | 326 | | |
328 | 327 | | |
329 | | - | |
| 328 | + | |
330 | 329 | | |
331 | 330 | | |
332 | 331 | | |
| |||
344 | 343 | | |
345 | 344 | | |
346 | 345 | | |
347 | | - | |
| 346 | + | |
348 | 347 | | |
349 | 348 | | |
350 | 349 | | |
| |||
362 | 361 | | |
363 | 362 | | |
364 | 363 | | |
365 | | - | |
| 364 | + | |
366 | 365 | | |
367 | 366 | | |
368 | 367 | | |
| |||
380 | 379 | | |
381 | 380 | | |
382 | 381 | | |
383 | | - | |
| 382 | + | |
384 | 383 | | |
385 | 384 | | |
386 | 385 | | |
| |||
469 | 468 | | |
470 | 469 | | |
471 | 470 | | |
472 | | - | |
473 | | - | |
474 | | - | |
475 | | - | |
476 | | - | |
477 | | - | |
478 | | - | |
479 | | - | |
| 471 | + | |
| 472 | + | |
| 473 | + | |
| 474 | + | |
| 475 | + | |
| 476 | + | |
| 477 | + | |
| 478 | + | |
| 479 | + | |
480 | 480 | | |
481 | | - | |
482 | | - | |
483 | | - | |
484 | | - | |
485 | | - | |
486 | | - | |
487 | | - | |
488 | | - | |
489 | | - | |
490 | | - | |
491 | | - | |
492 | | - | |
493 | | - | |
494 | | - | |
495 | | - | |
496 | | - | |
497 | | - | |
498 | | - | |
499 | | - | |
500 | | - | |
501 | | - | |
502 | | - | |
503 | | - | |
504 | | - | |
505 | | - | |
506 | | - | |
507 | 481 | | |
508 | 482 | | |
509 | 483 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
7 | 7 | | |
8 | 8 | | |
9 | 9 | | |
10 | | - | |
11 | | - | |
12 | 10 | | |
13 | 11 | | |
14 | 12 | | |
| |||
139 | 137 | | |
140 | 138 | | |
141 | 139 | | |
142 | | - | |
143 | | - | |
144 | | - | |
145 | 140 | | |
146 | 141 | | |
147 | | - | |
| 142 | + | |
148 | 143 | | |
149 | 144 | | |
150 | 145 | | |
| |||
330 | 325 | | |
331 | 326 | | |
332 | 327 | | |
333 | | - | |
334 | | - | |
335 | 328 | | |
336 | 329 | | |
337 | 330 | | |
| |||
341 | 334 | | |
342 | 335 | | |
343 | 336 | | |
344 | | - | |
| 337 | + | |
345 | 338 | | |
346 | 339 | | |
347 | 340 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
200 | 200 | | |
201 | 201 | | |
202 | 202 | | |
203 | | - | |
204 | 203 | | |
205 | 204 | | |
206 | 205 | | |
| |||
245 | 244 | | |
246 | 245 | | |
247 | 246 | | |
| 247 | + | |
| 248 | + | |
| 249 | + | |
| 250 | + | |
| 251 | + | |
248 | 252 | | |
249 | 253 | | |
250 | 254 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
7 | 7 | | |
8 | 8 | | |
9 | 9 | | |
10 | | - | |
| 10 | + | |
11 | 11 | | |
12 | 12 | | |
13 | 13 | | |
| |||
18 | 18 | | |
19 | 19 | | |
20 | 20 | | |
21 | | - | |
| 21 | + | |
22 | 22 | | |
23 | 23 | | |
24 | | - | |
25 | | - | |
26 | | - | |
| 24 | + | |
| 25 | + | |
| 26 | + | |
27 | 27 | | |
28 | 28 | | |
29 | 29 | | |
| |||
33 | 33 | | |
34 | 34 | | |
35 | 35 | | |
| 36 | + | |
36 | 37 | | |
37 | 38 | | |
38 | 39 | | |
| |||
59 | 60 | | |
60 | 61 | | |
61 | 62 | | |
62 | | - | |
63 | | - | |
| 63 | + | |
| 64 | + | |
64 | 65 | | |
65 | | - | |
66 | | - | |
| 66 | + | |
| 67 | + | |
67 | 68 | | |
68 | | - | |
| 69 | + | |
69 | 70 | | |
70 | 71 | | |
71 | 72 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
8 | 8 | | |
9 | 9 | | |
10 | 10 | | |
| 11 | + | |
| 12 | + | |
11 | 13 | | |
12 | 14 | | |
13 | 15 | | |
| |||
33 | 35 | | |
34 | 36 | | |
35 | 37 | | |
36 | | - | |
37 | | - | |
| 38 | + | |
| 39 | + | |
38 | 40 | | |
39 | 41 | | |
40 | 42 | | |
| |||
101 | 103 | | |
102 | 104 | | |
103 | 105 | | |
104 | | - | |
105 | | - | |
| 106 | + | |
| 107 | + | |
106 | 108 | | |
107 | 109 | | |
108 | 110 | | |
| |||
123 | 125 | | |
124 | 126 | | |
125 | 127 | | |
126 | | - | |
127 | | - | |
128 | | - | |
129 | | - | |
| 128 | + | |
| 129 | + | |
| 130 | + | |
| 131 | + | |
130 | 132 | | |
131 | 133 | | |
132 | 134 | | |
133 | 135 | | |
134 | | - | |
135 | | - | |
136 | | - | |
137 | | - | |
138 | | - | |
139 | | - | |
140 | | - | |
141 | | - | |
| 136 | + | |
| 137 | + | |
| 138 | + | |
| 139 | + | |
| 140 | + | |
| 141 | + | |
| 142 | + | |
| 143 | + | |
142 | 144 | | |
143 | 145 | | |
144 | | - | |
145 | | - | |
| 146 | + | |
| 147 | + | |
146 | 148 | | |
147 | 149 | | |
148 | 150 | | |
149 | 151 | | |
150 | 152 | | |
151 | 153 | | |
152 | | - | |
153 | | - | |
154 | | - | |
155 | | - | |
156 | | - | |
157 | | - | |
| 154 | + | |
| 155 | + | |
| 156 | + | |
| 157 | + | |
| 158 | + | |
| 159 | + | |
158 | 160 | | |
159 | 161 | | |
160 | 162 | | |
| |||
0 commit comments