Skip to content
GitLab
  • Menu
Projects Groups Snippets
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in
  • concrexit concrexit
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 70
    • Issues 70
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 10
    • Merge requests 10
  • Deployments
    • Deployments
    • Releases
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • Repository
  • Activity
  • Graph
  • Create a new issue
  • Commits
  • Issue Boards
Collapse sidebar
  • thalia
  • concrexitconcrexit
  • Merge requests
  • !137

Remove nodeJS from Dockerfile after moving to SASS

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Joost Rijneveld requested to merge remove-nodejs-dockerfile into master Oct 25, 2016
  • Overview 0
  • Commits 1
  • Changes 1

Omdat @sversteeg in !135 (merged) de dependency op less en nodeJS heeft opgeruimd is het ook niet meer nodig om dit in de Dockerfile op te nemen. De vervanger, libsass, komt gewoon via requirements.txt binnen en heeft dus geen extra aandacht nodig.

We hebben nu twee lijstjes met niet-pip-dependencies (namelijk in README.md en in Dockerfile). Ideeën hoe we van die duplicatie af kunnen zijn welkom. Gaat natuurlijk vanzelf als we de installatie-instructies vervangen door 'gebruik docker', wat we misschien op termijn wel willen doen.

Assignee
Assign to
Reviewer
Request review from
Time tracking
Source branch: remove-nodejs-dockerfile