Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bump toolkit and standard / nunjucks, and tidy package.json a bit #330

Merged
merged 3 commits into from
Dec 16, 2016

Conversation

robinwhittleton
Copy link
Contributor

This brings standard, nunjucks and govuk_frontend_toolkit up to the latest, and removes some older unused packages from package.json. Without a comprehensive test suite it’s always possible that removing stuff might be a problem, but:

  • hogan and consolidate seem to just be included for mustache rendering (which we’re not doing any more) and they’re aren’t manually included
  • minimist also isn’t included anywhere manually, but is a sub-dependency of express-prototype so doesn’t need to be expressly included

Robin Whittleton added 2 commits December 16, 2016 10:50
"marked": "^0.3.6",
"minimist": "0.0.8",
Copy link
Contributor

Choose a reason for hiding this comment

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

This isn't being used to set the port anymore, but looks like it might be useful, if we can keep it in - in case others are using it.

Copy link
Contributor

@gemmaleigh gemmaleigh left a comment

Choose a reason for hiding this comment

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

Minor comment about minimist, if we could leave that in then I'm happy to merge.

- consolidate was added to help replace mu2 for mustache rendering
- hogan.js was also in there for mustache rendering
@gemmaleigh gemmaleigh merged commit dc248f7 into master Dec 16, 2016
@gemmaleigh gemmaleigh deleted the bump-fet branch December 16, 2016 12:12
robinwhittleton pushed a commit that referenced this pull request Dec 16, 2016
- #330 Update GOV.UK toolkit and StandardJS to latest
- #328 Update GOV.UK template to latest
- #324 Fix the example question page’s back link
@robinwhittleton robinwhittleton mentioned this pull request Dec 16, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants