Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
  • B bootstrap
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 263
    • Issues 263
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 114
    • Merge requests 114
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and registries
    • Package Registry
    • Infrastructure Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • Bootstrap
  • bootstrap
  • Merge requests
  • !266

Make js/*.js pass jslint's checks; the existing JS broke when caching with Rails 3.0

  • Review changes

  • Download
  • Email patches
  • Plain diff
Closed Administrator requested to merge github/fork/purcell/master into master Sep 20, 2011
  • Overview 0
  • Commits 2
  • Pipelines 0
  • Changes 7

Created by: purcell

When concatenating or naively minifying the existing JS, the lack of semicolons resulted in ambiguous code and resulting breakage in one of my Rails apps. This commit makes the JS pass JSLint's checks, and the result is safer behavior. (Sorry that it necessarily touched so many lines.) Cheers! -Steve

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: github/fork/purcell/master