Skip to content

Add Cargo cooldown support#48

Merged
andrew merged 3 commits into
git-pkgs:mainfrom
silvenlily:cargo-cooldown
Apr 1, 2026
Merged

Add Cargo cooldown support#48
andrew merged 3 commits into
git-pkgs:mainfrom
silvenlily:cargo-cooldown

Conversation

@silvenlily
Copy link
Copy Markdown
Contributor

  • Added support for cooldowns for cargo
  • Added a test to test cooldowns with cargo

I don't typically write code in go (as you can probably guess I typically use rust) and I would very much appreciate it if someone would double check my work to make sure I didn't do anything stupid <3

- Added support for cooldowns for cargo
- Added a test to test cooldowns with cargo
add cargo to registry's with support for cooldowns
@silvenlily silvenlily marked this pull request as draft April 1, 2026 03:20
@silvenlily
Copy link
Copy Markdown
Contributor Author

I'm testing this live on an actual server with actual cargo and not actually seeing filtering

@silvenlily
Copy link
Copy Markdown
Contributor Author

Okay! It turns out I was just deeply stupid and put the wrong branch on my server, teehee

@silvenlily silvenlily marked this pull request as ready for review April 1, 2026 03:43
@andrew
Copy link
Copy Markdown
Contributor

andrew commented Apr 1, 2026

Thanks, I'll try to review this later today

Copy link
Copy Markdown
Contributor

@andrew andrew left a comment

Choose a reason for hiding this comment

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

I think we need a return here otherwise the code continues to run on after it.

Comment thread internal/handler/cargo.go
yep

Co-authored-by: Andrew Nesbitt <andrewnez@gmail.com>
@andrew andrew merged commit 922d44b into git-pkgs:main Apr 1, 2026
5 checks passed
@andrew
Copy link
Copy Markdown
Contributor

andrew commented Apr 1, 2026

Thanks @silvenlily, i'll tag a release at the end of the week that this will be included in.

@silvenlily
Copy link
Copy Markdown
Contributor Author

:)

@silvenlily silvenlily deleted the cargo-cooldown branch April 1, 2026 17:29
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.

2 participants