Details
-
Type:
Task
-
Status: Closed
-
Priority:
Minor
-
Resolution: Fixed
-
Affects Version/s: 2.0.3
-
Fix Version/s: 2.0.4
-
Component/s: General
-
Labels:None
-
Testing Instructions:
-
Affected Branches:MOODLE_20_STABLE
-
Fixed Branches:MOODLE_20_STABLE
-
Pull from Repository:
-
Pull Master Branch:admin_whitespace_cleanup_master
-
Pull Master Diff URL:
-
Rank:17399
Description
This is one simple cleanup of whitespace in the admin directory, both 20_STABLE and master branches.
This task has been done to be able to test some continuous integration tests and check that changes in whitespace are properly detected by the process.
So no much history beyond the change.
Activity
Integrated thanks Eloy
Show
Sam Hemelryk
added a comment - Integrated thanks Eloy
Thanks Sam, I did it, always both to 20_STABLE and master both to reduce some whitespace, but mainly to test that my integration tests are detecting correctly the changes in the number of incorrect whitespaces.
Surely I'll run some more iterations in the next weeks.
Show
Eloy Lafuente (stronk7)
added a comment - Thanks Sam, I did it, always both to 20_STABLE and master both to reduce some whitespace, but mainly to test that my integration tests are detecting correctly the changes in the number of incorrect whitespaces.
Surely I'll run some more iterations in the next weeks.
nothing really to test, just that all the changes are ok and no code is affected at all.
Show
Eloy Lafuente (stronk7)
added a comment - nothing really to test, just that all the changes are ok and no code is affected at all.
This is now upstream, yay! Many thanks!
Show
Eloy Lafuente (stronk7)
added a comment - This is now upstream, yay! Many thanks!
Looks fine to me
Once I've checked with you Eloy that things can be integrated this will go in.