[#45745] - [6.x] Typos action improvement
- Pending
- Medium
- Build: 6.0-dev
- # 45745
- Diff
- brianteeman:typos_improve
User tests: Successful: Unsuccessful:
Summary of Changes
small tweaks to the extend-exclude settings
Removed the exception for "/administrator/components/com_admin/sql/* as its no longer needed
Allowed typos to check the en-GB installation language files
Corrected the path for /includes/
Testing Instructions
Code review
Check action still passes
Link to documentations
Please select:
-
Documentation link for docs.joomla.org:
-
No documentation changes for docs.joomla.org needed
-
Pull Request link for manual.joomla.org:
-
No documentation changes for manual.joomla.org needed
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Repository |
| Labels |
Added:
PR-6.0-dev
|
||
@brianteeman Was there a reason for triggering a branch update (and so invalidating the human test counter with my test)?
yes - i was hoping it would be merged today and was trying to save someone some time as the merger always does an update before merge
Why should it be merged if it has only 1 (and now zero) tests?
I will now restore my test result in the issue tracker, but don't count on me doing that every time.
Not my fault then if PRs are not found when filtering in the issue tracker by the number of test results to see if there is a PR which needs one test, or to see if a PR has 2 and can be set RTC, if the counter is always reset.
I have tested this item ✅ successfully on 617bed8
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/45745.