remove

[github-actions]

bones/framework_crates/bones_framework/src/networking/online.rs

Lines 375 to 382 in 01823b4

	/// Search for game with `matchmaking_server` and `player_count`
	pub fn start_search_for_game(matchmaking_server: String, player_count: usize) {
	// TODO remove
	info!("Starting search for online game with player count {player_count}");
	ONLINE_MATCHMAKER
	.try_send(OnlineMatchmakerRequest::SearchForGame {
	addr: matchmaking_server.clone(),
	player_count,

---

This issue was generated by todo-issue based on a TODO comment in 01823b4.

[MaxCWhitehead]

lol @zicklag I need to be more careful about my TODO comments I guess, didn't know they'd open issues. Probably for the best as this way I actually will clean up the comments...

Feel free to assign any of these to me. (At least, the ones that are not real issues)

[erlend-sh]

Maybe there's some place in our codebase/docs where we could clearly point out that we've got some special handling in place for TODO comments?

[zicklag]

Yeah, we have a minor note about TODO comments in the README: https://github.com/fishfolk/bones?tab=readme-ov-file#contributing.

Honestly I probably should have caught this when I reviewed the code to prevent it from getting in in the first place. Either way, it's good that there are obvious so that we must go and remove them, I suppose. :)

[github-actions]

Closing Issue

This comment was generated by todo-issue based on the deletion of a TODO comment in 2f4e7ac.