Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

stencil outline test #395

Draft
wants to merge 7 commits into
base: master
Choose a base branch
from
Draft

Conversation

pollend
Copy link
Collaborator

@pollend pollend commented Sep 14, 2024

@pollend pollend marked this pull request as draft September 14, 2024 15:21
@pollend pollend force-pushed the feature/test-code-stencil-outline branch from 6476f2d to 5c3fe45 Compare September 15, 2024 04:41
@pollend pollend marked this pull request as ready for review September 17, 2024 23:33
@@ -230,6 +230,7 @@ void Com_Printf( const char *format, ... )
Q_vsnprintfz( msg, sizeof( msg ), format, argptr );
va_end( argptr );

printf("%s", msg);
Copy link
Collaborator

Choose a reason for hiding this comment

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

ah this probably wasn't supposed to get committed?.. although maybe we should have a flag for enabling console output ?

Signed-off-by: Michael Pollind <[email protected]>
Signed-off-by: Michael Pollind <[email protected]>
Signed-off-by: Michael Pollind <[email protected]>
Signed-off-by: Michael Pollind <[email protected]>
Signed-off-by: Michael Pollind <[email protected]>
@pollend pollend force-pushed the feature/test-code-stencil-outline branch from d497beb to 994ef7a Compare September 18, 2024 14:42
@pollend pollend changed the base branch from velzie/2.16 to master September 18, 2024 14:43
Signed-off-by: Michael Pollind <[email protected]>
@pollend pollend force-pushed the feature/test-code-stencil-outline branch from 404d54d to 84b16ad Compare September 19, 2024 02:28
@pollend
Copy link
Collaborator Author

pollend commented Sep 19, 2024

@pollend
Copy link
Collaborator Author

pollend commented Oct 31, 2024

umm seems like there are problems on some machines with this. umm I feel like maybe i rather do this in the NRI branch.

@velzie
Copy link
Collaborator

velzie commented Nov 1, 2024

yeah i think it's fine if it ends up being nri exclusive. whatever is easier to write and maintain

@pollend pollend marked this pull request as draft November 1, 2024 21:56
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