Skip to content
This repository has been archived by the owner on May 4, 2022. It is now read-only.

Simplify build #335

Merged
merged 8 commits into from
Oct 13, 2016
Merged

Simplify build #335

merged 8 commits into from
Oct 13, 2016

Conversation

jsamsa
Copy link
Contributor

@jsamsa jsamsa commented Oct 11, 2016

This change was to move functionality found in build.sh files and Grunt configuration into the npm build with the goal of simplifying the build process by reducing the number of tools required.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 39.537% when pulling 607897b on jsamsa:simplify-build into 230549a on UW-Madison-DoIT:master.

@vertein
Copy link
Contributor

vertein commented Oct 12, 2016

Copy link
Contributor

@timlevett timlevett left a comment

Choose a reason for hiding this comment

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

Looks good, I would also recommend updating the documentation on the quickstart and readme.md to reflect the different npm run commands.

Copy link
Contributor

@timlevett timlevett left a comment

Choose a reason for hiding this comment

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

The test cases are failing with error sh: 1: rimraf: not found. Please fix.

@jsamsa
Copy link
Contributor Author

jsamsa commented Oct 12, 2016

  • Fixed missing rimraf issue
  • Updated documentation
  • Modified Dockerfile

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.4%) to 39.157% when pulling 201d67e on jsamsa:simplify-build into 230549a on UW-Madison-DoIT:master.


RUN apt-get update && apt-get install -y bzip2 && apt-get install -y git

# Add frame
COPY tools /build/tools
copy docs /build/docs
Copy link
Contributor

Choose a reason for hiding this comment

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

uppercase COPY is more conventional

@paulerickson
Copy link
Contributor

👍 less is more!

@timlevett timlevett merged commit ab764d6 into uPortal-Attic:master Oct 13, 2016
vertein pushed a commit to vertein/uportal-app-framework that referenced this pull request Jun 3, 2017
MUMUP-2149 : rename sample staticFeeds
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

5 participants