#8687 closed bug (fixed)
.jshintrc files not copied to dist
| Reported by: | southerd | Owned by: | mikesherov |
|---|---|---|---|
| Priority: | minor | Milestone: | 1.9.1 |
| Component: | [meta] ui.build | Version: | 1.9.0 |
| Keywords: | Cc: | ||
| Blocked by: | Blocking: |
Description
When building using grunt release, the .jshint files are not copied to the dist directory. This causes tests to fail when run against the build.
Change History (10)
comment:1 Changed 10 years ago by
| Owner: | set to southerd |
|---|---|
| Status: | new → pending |
comment:2 Changed 10 years ago by
| Status: | pending → open |
|---|
mikesherov: Our QUnit tests run JSHint and they pull in the .jshintrc file using ajax. Of course, you can disable the JSHint test with ?nojshint=true, but we should probably have the default settings for tests working.
comment:3 Changed 10 years ago by
| Owner: | changed from southerd to mikesherov |
|---|---|
| Status: | open → assigned |
Got it. That's certainly interesting. Seems the path to the file just needs one more ../
Pull request coming shortly
comment:4 Changed 10 years ago by
That won't work- .jshint will then no longer work when run from the source directory (./test). The dist directory I referred to is dist/jquery-ui-<%= version %>/ which contains the concatenated and minified files. The underlying problem is that minimatch, the globbing library grunt uses, doesn't copy hidden . files in the 0.3.x grunt releases- Ben Alman will be fixing this in 0.4, but another solution might be needed now.
comment:5 Changed 10 years ago by
Yup, you're right. I just wrapped my head around it. you're talking about the test suite in dist. Looking for a solution now.
comment:6 Changed 10 years ago by
To save you a bit of time, I poked at it a bit during the Summit.
Changing the copy line from "ui/*" to "ui/{.,}*" does not seem to work.
Adding the line "ui/.jshintrc" puts the .jshint in the wrong location (in jquery-ui-{version}/ instead of jquery-ui-{version}/ui)
I regressed to hand-copying after a build- didn't have time to shave yaks at the summit :(
comment:9 Changed 10 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | assigned → closed |
Build: Include ui/.jshintrc. Fixes #8687 - .jshintrc files not copied to dist.
Changeset: 390510c7059d98d68cfe082530051c476ee08be4
comment:10 Changed 10 years ago by
| Milestone: | 1.10.0 → 1.9.1 |
|---|
The .jshintrc files aren't really needed in the distributed files, so we only copied the one that's required for the QUnit tests.
Thanks for contributing! I'm not able to reproduce this issue. .jshintrc is not being copied to the dist directory, but how would this be causing the tests to fail? Can you be more specific in what you're doing when you say "run against the build"?