search

cfpb / regulations-site

(DEPRECATED) Web interface for viewing U.S. federal regulations and other regulatory information
Other
28 stars 43 forks source link

updated README #699

Closed grapesmoker closed 9 years ago

grapesmoker commented 9 years ago

Updated the README to remind people that they need to specify the locations of their static resources before starting the demo server.

KimberlyMunoz commented 9 years ago

Is this related to #698?

I thought that was about running a grunt build or grunt squish to get the minified JS and avoid the empty sidebar.

grapesmoker commented 9 years ago

Is that how you're supposed to do it? I'm not too well-versed in Grunt, obviously. I will say though that when I run grunt build I get a mile-high stacktrace, so that's how I got around that problem.

grapesmoker commented 9 years ago

I'm getting:

1) Non-DOM Helper functions: "before all" hook: ReferenceError: Set is not defined (stacktrace follows)

2) Version Finder Helper Functions: "before all" hook: ReferenceError: Set is not defined (stacktrace follows)

No coverage information was collected, exit without writing coverage information

Warning: Task "mocha_istanbul:coverage" failed. Use --force to continue.

Aborted due to warnings.

KimberlyMunoz commented 9 years ago

Delete your node_modules folder and run npm install again.

grapesmoker commented 9 years ago

Just did that, still having same issue.

KimberlyMunoz commented 9 years ago

The node dependency issue that caused those errors is fixed in #700. Should be good after that.

ascott1 commented 9 years ago

@grapesmoker Should we merge this and then @KimberlyMunoz or I can update the grunt instructions?

grapesmoker commented 9 years ago

I think we should just update the grunt instructions. My edit was based on an erroneous understanding of what was going on.

On Fri, Mar 27, 2015 at 9:49 AM, Adam Scott notifications@github.com wrote:

@grapesmoker https://github.com/grapesmoker Should we merge this and then @KimberlyMunoz https://github.com/KimberlyMunoz or I can update the grunt instructions?

— Reply to this email directly or view it on GitHub https://github.com/cfpb/regulations-site/pull/699#issuecomment-86944914.

http://www.google.com/profiles/grapesmoker

ascott1 commented 9 years ago

Cool. Let's close this and I'll update those instructions.