Skip to content

Conversation

@Masterkatze
Copy link
Collaborator

No description provided.

@Masterkatze Masterkatze force-pushed the sniper branch 2 times, most recently from ce281f7 to dd9bd84 Compare March 11, 2025 17:14
@Masterkatze Masterkatze marked this pull request as ready for review March 11, 2025 22:02
Copy link

@brysondev brysondev left a comment

Choose a reason for hiding this comment

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

Nit: using namespace fmt;

Idk if its considered best practice anymore but, you typed :: a lot :^)

@Masterkatze
Copy link
Collaborator Author

Nit: using namespace fmt;

Idk if its considered best practice anymore but, you typed :: a lot :^)

For some reason I like writing std:: and don't like using namespace :)

@nullsystem
Copy link

Nit: using namespace fmt;

Idk if its considered best practice anymore but, you typed :: a lot :^)

using namespace sucks, only ok for literals

@Masterkatze Masterkatze merged commit 5c38c2d into NeotokyoRebuild:master Mar 11, 2025
1 check passed
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.

3 participants