Skip to content

Conversation

@brianteeman
Copy link
Contributor

I was testing a script using the joomla repo and found these typos/grammar errors.

The basic english language rule is that if you want to write "a something" and the "something" begins with a vowel then it changes to "an"

This also applies even if the "something" sounds like it begins with a vowel.

As these are all corrections in comments testing is just by review

I was testing a script using the joomla repo and found these typos/grammar errors.

The basic english language rule is that if you want to write "a something" and the "something" begins with a vowel then it changes to "an"

This also applies even if the "something" _sounds_ like it begins with a vowel.

As these are all corrections in comments testing is just by review
@joomla-cms-bot joomla-cms-bot added NPM Resource Changed This Pull Request can't be tested by Patchtester PR-4.2-dev Unit/System Tests labels Dec 16, 2022
@sandewt
Copy link
Contributor

sandewt commented Dec 16, 2022

I have tested this item ✅ successfully on 52d7ce0


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/39432.

1 similar comment
@Quy
Copy link
Contributor

Quy commented Dec 16, 2022

I have tested this item ✅ successfully on 52d7ce0


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/39432.

@Quy Quy removed Unit/System Tests NPM Resource Changed This Pull Request can't be tested by Patchtester labels Dec 16, 2022
@Quy
Copy link
Contributor

Quy commented Dec 16, 2022

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/39432.

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Dec 16, 2022
@joomla-cms-bot joomla-cms-bot added NPM Resource Changed This Pull Request can't be tested by Patchtester Unit/System Tests labels Dec 17, 2022
@roland-d roland-d merged commit 3cdd305 into joomla:4.2-dev Dec 17, 2022
@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Dec 17, 2022
@roland-d
Copy link
Contributor

Thank you

@roland-d roland-d added this to the Joomla! 4.2.7 milestone Dec 17, 2022
@richard67
Copy link
Member

@roland-d I know it’s only grammar in comments, but have you seen my review comment?

@brianteeman
Copy link
Contributor Author

@roland-d I know it’s only grammar in comments, but have you seen my review comment?

??

@brianteeman brianteeman deleted the grammar branch December 17, 2022 22:41
@roland-d
Copy link
Contributor

@richard67 I only see that you merged 4.2-dev branch into here. There is nothing else.

@richard67
Copy link
Member

Ah i see the review was pending.

@richard67
Copy link
Member

Can you see it now? Review was pending still, my mistake.

@roland-d
Copy link
Contributor

Now it is visible.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

NPM Resource Changed This Pull Request can't be tested by Patchtester Unit/System Tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants