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

Update Squeaky Clean instructions #2857

Merged
merged 4 commits into from
Nov 6, 2024

Conversation

jagdish-15
Copy link
Contributor

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
Contributor 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.

@@ -14,7 +14,8 @@
"SleeplessByte",
"Smarticles101",
"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
Contributor 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.

@@ -29,11 +29,14 @@ SqueakyClean.clean("a-bc");
## 3. Convert leetspeak to normal text

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
Contributor 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.

@@ -4,7 +4,8 @@
],
"contributors": [
"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
Contributor 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
Contributor 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