Skip to content

Conversation

@jagdish-15
Copy link
Member

pull request

This pull request is aimed at changing the instructions.md for the Squeaky-Clean exercise in the track.
Ref- https://forum.exercism.org/t/java-squeaky-clean-unclear-readme/13575


Reviewer Resources:

Track Policies

Copy link
Member

Choose a reason for hiding this comment

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

Did you mean to update this file? It’s for a different exercise so that’s out of scope most likely.

Copy link
Member Author

Choose a reason for hiding this comment

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

I apologize for the confusion. I made this change in a different branch, and I'm not sure how it ended up here. Any suggestions you have would be greatly appreciated. I've learned from this mistake, and I assure you it won't happen again.

"sshine",
"Wasy18"
"Wasy18",
"jagdishdrp"
Copy link
Member

Choose a reason for hiding this comment

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

I think this is changing in one of your other PRs. Could you remove this change from this PR?

Copy link
Member Author

Choose a reason for hiding this comment

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

I’m not entirely sure how this ended up here, and I apologize for the oversight. I promise it won’t happen again.


Modify the (_static_) `SqueakyClean.clean()` method to convert [leetspeak][leet-speak] to normal text.

For simplicity we will only be using `4`, `3`, `0`, `1` and `7` from the table.
Copy link
Member

Choose a reason for hiding this comment

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

The following suggestion is from the discussion in the forum post:

Suggested change
For simplicity we will only be using `4`, `3`, `0`, `1` and `7` from the table.
For simplicity we will only be replacing 4, 3, 0, 1 and 7 with a, e, o, l, and t, respectively.

Copy link
Member Author

Choose a reason for hiding this comment

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

I have updated this line as suggested by Mr. Donogheue on the forum. I apologize for the earlier mistake. Please let me know if there's anything else you would like me to change.

"sanderploegsma",
"manumafe98"
"manumafe98",
"jagdishdrp"
Copy link
Member

Choose a reason for hiding this comment

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

This should be your Github handle. I'd also like to add mrDonoghue here since they first suggested this change in the forums.

Copy link
Member Author

@jagdish-15 jagdish-15 Nov 5, 2024

Choose a reason for hiding this comment

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

I am sorry for the oversight. I have added mrDonoghue and arranged the handles in alphabetical order, which was not maintained previously (even before this PR).

Copy link
Member

@kahgoh kahgoh left a comment

Choose a reason for hiding this comment

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

All good, no need to apologize @jagdish-15 ;)

Thanks for making the updates!

@kahgoh kahgoh changed the title Update instructions squeaky Update Squeaky Clean instructions Nov 6, 2024
@kahgoh kahgoh merged commit 1eec3d1 into exercism:main Nov 6, 2024
2 checks passed
@jagdish-15
Copy link
Member Author

Thank you so much, @kahgoh! If you come across any contributions that would be a good fit for me, please feel free to tag me on the forum. I’d be glad to work on them and submit a PR!

@jagdish-15 jagdish-15 deleted the update-instructions-squeaky branch November 6, 2024 13:05
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