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

List of exceptions that could have their messages improved #17

Open
upasana-me opened this issue Jun 26, 2013 · 11 comments
Open

List of exceptions that could have their messages improved #17

upasana-me opened this issue Jun 26, 2013 · 11 comments
Labels

Comments

@upasana-me
Copy link
Collaborator

@upasana-me upasana-me commented Jun 26, 2013

No description provided.

@sartak
Copy link
Owner

@sartak sartak commented Jun 26, 2013

CoercingWithoutCoercions

@upasana-me
Copy link
Collaborator Author

@upasana-me upasana-me commented Jun 27, 2013

AccessorMustReadWrite

@upasana-me
Copy link
Collaborator Author

@upasana-me upasana-me commented Jul 6, 2013

CannotDelegateWithoutIsa

@upasana-me
Copy link
Collaborator Author

@upasana-me upasana-me commented Jul 7, 2013

I think messages of CannotOverrideLocalMethodIsPresent & CannotAugmentIfLocalMethodPresent needs improvement, but not sure.

@upasana-me
Copy link
Collaborator Author

@upasana-me upasana-me commented Jul 26, 2013

BadOptionFormat

@upasana-me
Copy link
Collaborator Author

@upasana-me upasana-me commented Aug 9, 2013

OverrideMethodConflictInRoleComposition, concerned method name should be in the exception message

@upasana-me
Copy link
Collaborator Author

@upasana-me upasana-me commented Aug 20, 2013

InvalidArgumentToMethod

@upasana-me
Copy link
Collaborator Author

@upasana-me upasana-me commented Sep 14, 2013

Message of AttributeConflictInSummation has some extra space

@upasana-me
Copy link
Collaborator Author

@upasana-me upasana-me commented Sep 15, 2013

InvalidTypeConstraint

@upasana-me
Copy link
Collaborator Author

@upasana-me upasana-me commented Sep 21, 2013

Is this the next step?

Changing messages of the above exceptions, run test suite & see if any dependents' tests are failing, if yes, then fix them.

@sartak
Copy link
Owner

@sartak sartak commented Sep 21, 2013

I think this should wait until after your branch is released, actually. There's no rush on this, and the more potential breakage we could defer, the better.

iPhone�$B$+$iAw?.�(B

2013/09/21 11:36�$B!"�(BUpasana Shukla notifications@github.com �$B$N%a%C%;!<%8�(B:

Is this the next step?

Changing messages of the above exceptions, run test suite & see if any dependents' tests are failing, if yes, then fix them.

�$B!=�(B
Reply to this email directly or view it on GitHub.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
You can’t perform that action at this time.