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.

Sign up for GitHub

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

Jump to bottom

Pad should throw ArgumentOutOfRangeException for invalid width #1027

Closed
atifaziz opened this issue Oct 28, 2023 · 0 comments · Fixed by #1030
Closed

Pad should throw ArgumentOutOfRangeException for invalid width #1027

atifaziz opened this issue Oct 28, 2023 · 0 comments · Fixed by #1030
Assignees
@atifaziz
Milestone
4.0.0

Comments

@atifaziz
Copy link
Member

Pad should throw ArgumentException when the width argument is invalid. Is should throw ArgumentOutOfRangeException instead.

This is a compatible change.
@atifaziz atifaziz added this to the 4.0 milestone Oct 28, 2023
@atifaziz atifaziz self-assigned this Oct 28, 2023
This was referenced Oct 28, 2023
Closed
Merged
atifaziz added a commit that referenced this issue Oct 28, 2023
@atifaziz
Fix exception thrown for invalid "Pad"/"PadStart" width
092a40d 
This is a squashed merge of PR #1030 that:

- fixes #1027
- fixes #1028
julienasp pushed a commit to julienasp/MoreLINQ that referenced this issue Nov 14, 2023
@atifaziz @julienasp
Fix exception thrown for invalid "Pad"/"PadStart" width
de20099 
This is a squashed merge of PR morelinq#1030 that:

- fixes morelinq#1027
- fixes morelinq#1028
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@atifaziz atifaziz

Labels
None yet
Projects
None yet
Milestone
4.0.0
Development

Successfully merging a pull request may close this issue.

Fix exception thrown when Pad/PadStart width is invalid atifaziz/MoreLINQ
1 participant
@atifaziz