Skip to content

Refactoring output print to message #377

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

Merged
merged 1 commit into from
Jun 10, 2025

Conversation

roulpriya
Copy link
Contributor

@roulpriya roulpriya commented Jun 10, 2025

Fixes Issue: #378

To control printing user facing messages to stdout or stderr created a new function message to internally delegate to print after formatting.

In a subsequent change we will control the output to stdout or stderr.

@cmcgee1024
Copy link
Member

@roulpriya this looks good to me. There's a format check failure with an instruction on how to fix up the formatting:

1/60 files require formatting.
Please run 'swift run swiftformat .' to format the source code.

Once you update the formatting I can approve and merge this.

@roulpriya roulpriya force-pushed the change-print-output branch from 204ce66 to 9395b1d Compare June 10, 2025 16:23
@roulpriya
Copy link
Contributor Author

I fixed formatting @cmcgee1024

@cmcgee1024 cmcgee1024 merged commit ab6757f into swiftlang:main Jun 10, 2025
24 checks 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.

2 participants