cordlesscoder cordlesscoder
0 followers · 0 following
  • Joined on 2025-09-11
Repositories
1
 Projects Packages Code Public activity Starred repositories
1
cordlesscoder commented on pull request Skynet/discord-bot#46 2025-09-12 12:09:10 +00:00
Refactors for less nesting, slight performance and memory improvement for count command

Could you selectively merge whatever changes you're fine with? I'd like to address the ones I didn't get right in separate, smaller PRs

cordlesscoder commented on pull request Skynet/discord-bot#46 2025-09-12 12:04:16 +00:00
Refactors for less nesting, slight performance and memory improvement for count command

No it cannot?

cordlesscoder commented on pull request Skynet/discord-bot#46 2025-09-12 11:57:27 +00:00
Refactors for less nesting, slight performance and memory improvement for count command

But this is the entire point of shadowing existing - it is absolutely idiomatic Rust.

cordlesscoder created pull request Skynet/discord-bot#46 2025-09-11 12:39:45 +00:00
Refactors for less nesting, slight performance and memory improvement for count command
cordlesscoder created branch #46_Refactor in cordlesscoder/discord-bot 2025-09-11 12:39:20 +00:00
cordlesscoder pushed to #46_Refactor at cordlesscoder/discord-bot 2025-09-11 12:39:20 +00:00
f3f08962a4 Refactors for less nesting, slight performance and memory improvement for count command
6353d77360 Merge pull request 'Remove RwLock for database' (#45) from cordlesscoder/discord-bot:#45_Remove_RwLock into main
Compare 2 commits »
cordlesscoder created pull request Skynet/discord-bot#45 2025-09-11 11:55:28 +00:00
Remove RwLock for database
cordlesscoder created branch #45_Remove_RwLock in cordlesscoder/discord-bot 2025-09-11 11:55:09 +00:00
cordlesscoder pushed to #45_Remove_RwLock at cordlesscoder/discord-bot 2025-09-11 11:55:09 +00:00
062f826d28 Remove RwLock for database
d8f785b0db Merge pull request 'Fix typos' (#44) from cordlesscoder/discord-bot:#44_Typos into main
Compare 2 commits »
cordlesscoder created pull request Skynet/discord-bot#44 2025-09-11 11:37:30 +00:00
Fix typos
cordlesscoder pushed to #44_Typos at cordlesscoder/discord-bot 2025-09-11 11:36:51 +00:00
d70a037057 Blame: ignore Fix Typos commit
7e90f45196 Fix typos
Compare 2 commits »
cordlesscoder created branch #44_Typos in cordlesscoder/discord-bot 2025-09-11 11:22:04 +00:00
cordlesscoder pushed to #44_Typos at cordlesscoder/discord-bot 2025-09-11 11:22:04 +00:00
cordlesscoder pushed to main at cordlesscoder/discord-bot 2025-09-11 11:18:10 +00:00
cde3e93565 Various refactors
cordlesscoder closed pull request Skynet/discord-bot#43 2025-09-11 11:17:59 +00:00
Update dependencies, remote unnecessary RwLock on database, fix typos
cordlesscoder commented on pull request Skynet/discord-bot#43 2025-09-11 11:06:55 +00:00
Update dependencies, remote unnecessary RwLock on database, fix typos

The logic is identical - in what way is this "improper", it's just less nested.

cordlesscoder pushed to main at cordlesscoder/discord-bot 2025-09-11 10:59:05 +00:00
e5a40261d9 Restore Cargo.toml
cordlesscoder pushed to main at cordlesscoder/discord-bot 2025-09-11 10:21:33 +00:00
a56b7cbd3f Remove panics on invalid env arguments
cordlesscoder commented on pull request Skynet/discord-bot#43 2025-09-11 10:19:38 +00:00
Update dependencies, remote unnecessary RwLock on database, fix typos

It wasn't broken, it was simply missing support for many SVG features - maybe the logos it deals with don't use them, in which case it makes no difference. But the change is like 10 lines of…

cordlesscoder pushed to main at cordlesscoder/discord-bot 2025-09-11 10:10:54 +00:00
a6ce057030 Add Taplo configuration