mahara-contributors team mailing list archive

[Mahara Bot <1495364@xxxxxxxxxxxxxxxxxx>]

Reviewed:  https://reviews.mahara.org/5308
Committed: https://git.nzoss.org.nz/mahara/mahara/commit/c44d4bfee6380b65ec5471cc294740026aa65584
Submitter: Aaron Wells (aaronw@xxxxxxxxxxxxxxx)
Branch:    master

commit c44d4bfee6380b65ec5471cc294740026aa65584
Author: Robert Lyon <robertl@xxxxxxxxxxxxxxx>
Date:   Fri Sep 18 09:38:19 2015 +1200

Bug 1495364: Remove more compiled CSS from repo

Devs will now need to compile the CSS themeselves.

behatnotneeded

Change-Id: I2337143c793baf1330f860ba246fb295ebcbb702
Signed-off-by: Robert Lyon <robertl@xxxxxxxxxxxxxxx>

-- 
You received this bug notification because you are a member of Mahara
Contributors, which is subscribed to Mahara.
Matching subscriptions: Subscription for all Mahara Contributors -- please ask on #mahara-dev or mahara.org forum before editing or unsubscribing it!
https://bugs.launchpad.net/bugs/1495364

Title:
  Remove compiled CSS from git repository

Status in Mahara:
  In Progress

Bug description:
  With Bug 1465107, the CSS in the core themes is now actually compiled
  from a series of SASS files.

  When we initially started on this, I made the decision to keep the
  compiled CSS in the git repository because I thought it would make
  life easier for our low-tech folks, because they wouldn't have to
  compile the CSS themselves in order to get Mahara to run. And it has
  had this result, but it has also caused immense problems for us devs
  because every time we make a commit that touches one of the SCSS
  files, it gets a merge conflict and cannot be merged cleanly into
  Gerrit.

  Best practice with build products is to exclude them from the gerrit
  repository, and now I'm starting to understand why. ;) So we've
  decided to go ahead and remove the compiled CSS from the gerrit repo.
  But we still want to try to make it easy for people to run Mahara from
  git, without having to understand much about SASS.

  So the plan is:

  1. Add the compiled CSS to the .gitignore file and remove it from the repo.
  2. Add a step to Mahara's makefile, which does everything necessary to build the CSS.
  3. Set up the Jenkins test server to build the CSS before it runs its tests.
  4. Add the build CSS step to the deployment scripts (so that the Mahara zip files for download will still contain compiled CSS)
  5. For people who can't get the CSS build steps to work locally, we set up a web service on one of our servers, which allows you to download the compiled CSS for a given Mahara commit from Gerrit. Something like https://mahara.org/local/getcss.php?id=ae897fd
  6. We add another build step to Mahara's makefile, which detects the current commit, downloads the compiled CSS from the server, and puts it in the right place

To manage notifications about this bug go to:
https://bugs.launchpad.net/mahara/+bug/1495364/+subscriptions