Skip to content

Update Class name for RaceTrack#2051

Merged
mirkoperillo merged 1 commit into
exercism:mainfrom
cdpaiva:patch-1
Nov 3, 2021
Merged

Update Class name for RaceTrack#2051
mirkoperillo merged 1 commit into
exercism:mainfrom
cdpaiva:patch-1

Conversation

@cdpaiva
Copy link
Copy Markdown
Contributor

@cdpaiva cdpaiva commented Oct 29, 2021

Class name in instructions.md is not consistent with the Class name in the src file. This may be a little confusing when following the instructions. I changed it to RaceTrack to match the src file.


Reviewer Resources:

Track Policies

Class name in instructions.md is not consistent with the Class name in the src file. This may be a little confusing when following the instructions. I changed it to RaceTrack to match the src file.
@ericbalawejder
Copy link
Copy Markdown
Contributor

ericbalawejder commented Nov 1, 2021

@mirkoperillo Does this failed linter test reference line 32 of https://github.com/exercism/java/blob/main/config.json?
I'm not sure what the issue is? PR #2052 has the same error so I'm not sure if this is just the linter confiuration?

@mirkoperillo
Copy link
Copy Markdown
Contributor

@ericbalawejder it seems we miss some new constaint of configlet.
I agree with you, this is related to configuration and not PRs

Copy link
Copy Markdown
Contributor

@mirkoperillo mirkoperillo left a comment

Choose a reason for hiding this comment

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

LGTM

@mirkoperillo mirkoperillo merged commit acfdd2b into exercism:main Nov 3, 2021
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.

3 participants