Skip to content
This repository has been archived by the owner on Dec 10, 2021. It is now read-only.

PLAT-226 move gems #241

Merged
merged 11 commits into from
May 16, 2016
Merged

PLAT-226 move gems #241

merged 11 commits into from
May 16, 2016

Conversation

bimsonz
Copy link
Contributor

@bimsonz bimsonz commented May 13, 2016

@pvhee
Copy link
Contributor

pvhee commented May 13, 2016

@bendodd do you see an issue with the gemfile moving location (from within the theme, to the root) - ultimately we want the drupal files to be in their own dir (campaign/campaign) and docs, gemfiles, and what not, at the root of this repo probably \cc @bimsonz

@@ -1 +0,0 @@
2.1.5
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why remove this?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

because it now lives within the root and is gitignored, any specific reason we need it in the repo ?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's good to have it in the repo as rvm/rbenv respect it and automatically switches users to that version. I.e. if people have multiple projects with different versions they don't have to manually switch

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sweet, its reinstated

@@ -43,7 +43,7 @@ module.exports = function (grunt) {

shell: {
styleguide: {
command: 'node_modules/kss/bin/kss-node --source profiles/cr/themes/custom/campaign_base/sass/ --css ../css/styles.css --verbose --title "Comic Relief PatternLab"'
command: 'node_modules/kss/bin/kss-node --source profiles/cr/themes/custom/campaign_base/sass/ --destination profiles/cr/themes/custom/campaign_base/styleguide --css ../css/styles.css --verbose --title "Comic Relief PatternLab"'
Copy link
Contributor Author

@bimsonz bimsonz May 13, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@gusliedke \FYI added destination param

@bimsonz
Copy link
Contributor Author

bimsonz commented May 16, 2016

@gusliedke merge away

@gusliedke gusliedke merged commit a9957e5 into develop May 16, 2016
@gusliedke gusliedke deleted the feature/PLAT-226_move_gems branch May 16, 2016 13:59
@pvhee pvhee mentioned this pull request Jan 5, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants