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

Spelling in default comment #4272

Closed
grpistoia opened this issue Aug 12, 2019 · 5 comments

Comments

@grpistoia
Copy link

commented Aug 12, 2019

Hi

There is a spelling mistake in the default comment of a class, on the word "Responsability" should be spelled "Responsibility".

Is there any spell checker in Smalltalk? if so, maybe suggest it there.

Maybe suggest an example of a class well documented.

Great improvements in the last versions, good work people!

Thanks lots.

@welcome

This comment has been minimized.

Copy link

commented Aug 12, 2019

Thanks for opening your first issue! Please check the CONTRIBUTING documents for some tips about which information should be provided. You can find information of how to do a Pull Request here: https://github.com/pharo-project/pharo/wiki/Contribute-a-fix-to-Pharo

GitHub
The Sources for Pharo. Contribute to pharo-project/pharo development by creating an account on GitHub.

@MarcusDenker MarcusDenker added the Easy label Aug 12, 2019

@MarcusDenker MarcusDenker added this to Very Easy in Simple Issues for Beginners via automation Aug 12, 2019

@MarcusDenker

This comment has been minimized.

Copy link
Member

commented Aug 21, 2019

the class comment spelling error has already been fixed in Pharo8. But there where other cases, I add a PR for that (and might do a backport for just the comment fix)

@MarcusDenker

This comment has been minimized.

Copy link
Member

commented Aug 21, 2019

PR for Pharo8: #4376

@MarcusDenker

This comment has been minimized.

Copy link
Member

commented Aug 21, 2019

I checked Pharo7: it is fixed there, to. So now I wonder: where did you see it?

@MarcusDenker

This comment has been minimized.

Copy link
Member

commented Aug 21, 2019

merged the PR

Simple Issues for Beginners automation moved this from Very Easy to Done Aug 21, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
2 participants
You can’t perform that action at this time.