8000 Remove unnecessary .js files by lydell · Pull Request #4083 · jashkenas/coffeescript · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Remove unnecessary .js files #4083

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

Merged
merged 1 commit into from
Sep 3, 2015

Conversation

lydell
Copy link
Collaborator
@lydell lydell commented Sep 3, 2015

The example code in documentation/coffee/.coffee accidentally had duplicate
compiled .js versions committed: Both as documentation/js/
.js and as
documentation/coffee/*.js.

The former is generated when running cake doc:site, while the latter has no
cake task to be generated. Removing the former and then re-compiling index.html
results in all the code examples being missing in the resulting HTML, while
removing the latter makes no difference.

In conclusion, documentation/coffee/*.js must be the unnecessary duplicates.
They are removed by this commit.

The example code in documentation/coffee/*.coffee accidentally had duplicate
compiled .js versions committed: Both as documentation/js/*.js and as
documentation/coffee/*.js.

The former is generated when running `cake doc:site`, while the latter has no
cake task to be generated. Removing the former and then re-compiling index.html
results in all the code examples being missing in the resulting HTML, while
removing the latter makes no difference.

In conclusion, documentation/coffee/*.js must be the unnecessary duplicates.
They are removed by this commit.
@lydell lydell mentioned this pull request Sep 3, 2015
jashkenas added a commit that referenced this pull request Sep 3, 2015
@jashkenas jashkenas merged commit eb6baf2 into jashkenas:master Sep 3, 2015
@lydell lydell deleted the remove-unnecessary-files branch September 3, 2015 20:56
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
0