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
Changes required by adapt_authoring project #450
Conversation
This includes: - new 'grunt server-build' task, which allows building the framework in preparation for adding in the actual course JSON at a later time - two new switch options: --theme - used to specify an Adapt theme --outputdir - used to specify a directory where the /build folder will be created, this is used in outputting to different folders, what would be different framework 'builds'
This was causing build failures in Travis.
+1 |
I'm pretty sure Did you want to get the last character? if so Also should Apart from that it looks ok. |
Corrected per the comments from @cajones.
This is to allow 'grunt build' to run successfully when the 'production' switch is used with npm install.
|
||
if (outputdir) { | ||
if (outputdir.substring(outputdir.length - 1, outputdir.length) !== '/') { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
don't need the second argument to substring here
+1 |
1 similar comment
+1 |
Changes required by adapt_authoring project
Highlights are:
These settings will have no impact on existing grunt tasks.