Comments (5)
Are you referring to this case?
define( [
'module1',
'module2'
], function(module1) {
// I don't have a module2 argument
});
or this case?
define([
'module1',
'module2'
], function(module1, module2) {
module1();
// I don't use module2
});
from bullsfirst-jquery-backbone.
Oh, I see it doing the second one, and neither grunt nor sublime's jshint is catching it. That seems new. I think I was getting the unused errors very recently.
from bullsfirst-jquery-backbone.
Ah, and here's what I just observed: if the unused argument is at the END of the arguments list, JSHint catches it, but not if it's anywhere else in the arguments list.
And here, I found it's intentional JSHint behavior: jshint/jshint@5751c5e
from bullsfirst-jquery-backbone.
Thanks for tracking this down. So the commit happened 5 months ago - does this mean that it has not made it to grunt?
from bullsfirst-jquery-backbone.
No, this is the intended behavior. Unuseds that are followed by useds will not throw an error.
That commit message explains why. I think it should be an optional flag, but that's what we have.
from bullsfirst-jquery-backbone.
Related Issues (20)
- Trade Dialog - Order estimates fetched twice from the server HOT 2
- Trade Dialog - Validation has not been implemented
- Accounts Page - Implement Add Account
- Orders Page - Reset Button Functionality
- Orders & Transactions Pages - Date Picker Stays Open
- Orders Page - Styling Drop Down HOT 2
- Transfer Modal - Toggle Button Resizing & Hover Fade HOT 1
- Trade Modal - No Dynamic Error Handling HOT 1
- Transfer Modal - Error Handling Messages Stay Open
- Tab Titles - Shift Down
- All Screens - Drop Downs / Text Fields / Date Fields Text Padding HOT 1
- Orders Page - Table Data w/ Really Long Account Names HOT 4
- Alert Boxes - Styling Changes
- Transactions Page - Reset Button Functionality HOT 1
- Transfer Modal - Styling Drop Down
- Trade Modal - Account Selection Drop Down HOT 1
- All Screens - Remove Underline from Action Links
- Switching quickly to positions tab crashes the application
- Transfer Modal - Replace serializeForm with toObject
- Selectboxes do not show a scrollbar in the dropdown for longer lists
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from bullsfirst-jquery-backbone.