Skip to content

Added less confusing whitespace error message - #305

Merged
charliemirabile merged 1 commit into
underground-software:masterfrom
mmyers-eng:specific-acceptance-message
Dec 2, 2025
Merged

Added less confusing whitespace error message#305
charliemirabile merged 1 commit into
underground-software:masterfrom
mmyers-eng:specific-acceptance-message

Conversation

@mmyers-eng

Copy link
Copy Markdown
Contributor

The submission is still accepted if whitespace errors are present.

@charliemirabilecharliemirabile left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks, I agree that this is confusing, but I have a different suggestion for how to improve it.

Comment threadorbit/radius.py Outdated
@mmyers-eng
mmyers-engforce-pushed the specific-acceptance-message branch from 41e9857 to 0653c9fCompareDecember 2, 2025 02:40
The submission is still accepted if whitespace errors are present.
Signed-off-by: Matt Myers <nt3058@gmail.com>
@mmyers-eng
mmyers-engforce-pushed the specific-acceptance-message branch from 0653c9f to d71797aCompareDecember 2, 2025 02:53

@charliemirabilecharliemirabile left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@charliemirabile
charliemirabile merged commit 70d3197 into underground-software:masterDec 2, 2025
1 of 4 checks passed
Sign up for freeto 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

@mmyers-eng@charliemirabile