delinted #524

Merged
merged 1 commit into from Sep 18, 2012

Conversation

Projects
None yet
2 participants
Contributor

drewfish commented Sep 18, 2012

No description provided.

Contributor

drewfish commented Sep 18, 2012

travis build fails -after- mojito jslint succeeds

Collaborator

caridy commented Sep 18, 2012

deja-vu, I think I reviewed this commit few days ago. Anyway:

  • why are we using "stupid" flag?
  • Y.mojito.util.shouldShareYUIInstance(opts.appShareYUIInstance, this.instance); is either not needed or should be assigned to something I believe.

Still:

+1

Contributor

drewfish commented Sep 18, 2012

Yeah, this is a redo of PR #509.

Contributor

drewfish commented Sep 18, 2012

Without /*stupid:true*/ I get Unexpected sync method errors.

@drewfish drewfish added a commit that referenced this pull request Sep 18, 2012

@drewfish drewfish Merge pull request #524 from drewfish/perf-jslint
delinted
39ed99b

@drewfish drewfish merged commit 39ed99b into YahooArchive:develop-perf Sep 18, 2012

1 check failed

default The Travis build failed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment