Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-27723

Cleanup incorrect whitespace in admin directory

    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

      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.

        Gliffy Diagrams

          Attachments

            Activity

            Hide
            samhemelryk Sam Hemelryk added a comment -

            Looks fine to me
            Once I've checked with you Eloy that things can be integrated this will go in.

            Show
            samhemelryk Sam Hemelryk added a comment - Looks fine to me Once I've checked with you Eloy that things can be integrated this will go in.
            Hide
            samhemelryk Sam Hemelryk added a comment -

            Integrated thanks Eloy

            Show
            samhemelryk Sam Hemelryk added a comment - Integrated thanks Eloy
            Hide
            stronk7 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.

            Show
            stronk7 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.
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            nothing really to test, just that all the changes are ok and no code is affected at all.

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - nothing really to test, just that all the changes are ok and no code is affected at all.
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            This is now upstream, yay! Many thanks!

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - This is now upstream, yay! Many thanks!

              People

              • Assignee:
                stronk7 Eloy Lafuente (stronk7)
                Reporter:
                stronk7 Eloy Lafuente (stronk7)
                Integrator:
                Sam Hemelryk
                Tester:
                Eloy Lafuente (stronk7)
                Participants:
              • Votes:
                0 Vote for this issue
                Watchers:
                0 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  1/Aug/11