Skip to content

Update Class name for RaceTrack #2051

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

Merged
merged 1 commit into from
Nov 3, 2021
Merged

Update Class name for RaceTrack #2051

merged 1 commit into from
Nov 3, 2021

Conversation

cdpaiva
Copy link
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
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
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
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