Skip to content

Remove '(static)' in default exception messages for non-static method…#2351

Merged
sanderploegsma merged 1 commit into
exercism:mainfrom
joe-mccann-dev:update-elonstoycar
Sep 1, 2023
Merged

Remove '(static)' in default exception messages for non-static method…#2351
sanderploegsma merged 1 commit into
exercism:mainfrom
joe-mccann-dev:update-elonstoycar

Conversation

@joe-mccann-dev
Copy link
Copy Markdown
Contributor

…s, as represented in ElonsToyCarTest.java.

pull request

This pull request revises the stub's exception messages to match behavior in the test file. Three methods were referred to as "static" but are instance methods in the test file.


Reviewer Resources:

Track Policies

Copy link
Copy Markdown
Contributor

@sanderploegsma sanderploegsma left a comment

Choose a reason for hiding this comment

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

LGTM, thanks 👍

@sanderploegsma sanderploegsma added the x:size/tiny Tiny amount of work label Sep 1, 2023
@sanderploegsma sanderploegsma merged commit 9414ddf into exercism:main Sep 1, 2023
@joe-mccann-dev joe-mccann-dev deleted the update-elonstoycar branch September 4, 2023 17:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

x:size/tiny Tiny amount of work

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants