Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
  • G gulp
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 24
    • Issues 24
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 3
    • Merge requests 3
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and registries
    • Package Registry
    • Infrastructure Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • gulp
  • gulp
  • Merge requests
  • !859

[recipes] Correct CSS pre-processors section

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Administrator requested to merge github/fork/donovanmuller/patch-1 into master Jan 08, 2015
  • Overview 12
  • Commits 1
  • Pipelines 0
  • Changes 1

Created by: donovanmuller

As someone new to Gulp and still finding my feet, I naively copy and pasted the code from the CSS pre-processors section thinking it would just work (following the same conventions for directory layout as described in the simple example).

It does not just work however

  • the scss directory is relative to the base directory (which is not the convention set out in the previous example)
  • the sass tasks paths also similarly do not follow convention
  • running just grunt serve as per the simple example does not invoke the sass task by default. (Of course you can run grunt sass serve but it seems more convenient if I could just run the same grunt serve as in the simple example - not sure if this is bad practice or not?)

Given the above, I've corrected the paths and add the sass task as a dependency to serve. I've also filled in the blanks just to make things as clear as possible for someone getting into Gulp. I.e. just copy, paste and run.

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: github/fork/donovanmuller/patch-1