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

(1036---clean-up-pillbox-lint-issues) cleans up lint warnings #1038

Conversation

cmcculloh-kr
Copy link

fixes #1036

@interactivellama
Copy link
Contributor

You're not // avoid confusion with $.attr();. That's why the array notation was used. If you are suggesting we use dot notation, who should probably say "NOT A JQUERY OBJECT" somewhere then.

@cmcculloh-kr
Copy link
Author

@interactivellama hey man, I'm not really sure what you're saying here.

Does my code not work? Where are you getting the // avoid confusion with $.attr(); from? I'm not seeing that anywhere...

interactivellama added a commit that referenced this pull request Feb 6, 2015
…lint-issues

(1036---clean-up-pillbox-lint-issues) cleans up lint warnings Fixes #1036.
@interactivellama interactivellama merged commit 38e00ee into ExactTarget:master Feb 6, 2015
@interactivellama interactivellama added this to the 3.6.0 milestone Feb 9, 2015
@cmcculloh-kr cmcculloh-kr deleted the 1036---clean-up-pillbox-lint-issues branch February 12, 2015 22:03
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.

Clean up pillbox.js lint issues
3 participants