Skip to content

Latest commit

 

History

History
200 lines (112 loc) · 7.1 KB

CONTRIBUTING.md

File metadata and controls

200 lines (112 loc) · 7.1 KB

Contributing to jQuery

  1. Getting Involved
  2. Discussion
  3. How To Report Bugs
  4. Core Style Guide
  5. Tips For Bug Patching

Getting Involved

There are a number of ways to get involved with the development of jQuery core. Even if you've never contributed code to an Open Source project before, we're always looking for help identifying bugs, writing and reducing test cases and documentation.

This is the best way to contribute to jQuery core. Please read through the full guide detailing How to Report Bugs.

Discussion

Forum and IRC

The jQuery core development team frequently tracks posts on the jQuery Development Forum. If you have longer posts or questions please feel free to post them there. If you think you've found a bug please file it in the bug tracker.

Additionally most of the jQuery core development team can be found in the #jquery-dev IRC channel on irc.freenode.net.

Weekly Status Meetings

Every week (unless otherwise noted) the jQuery core dev team has a meeting to discuss the progress of current work and to bring forward possible new blocker bugs for discussion.

The meeting is held in the #jquery-meeting IRC channel on irc.freenode.net at Noon EST on Mondays.

Past Meeting Notes

How to Report Bugs

Make sure it is a jQuery bug

Many bugs reported to our bug tracker are actually bugs in user code, not in jQuery code. Keep in mind that just because your code throws an error and the console points to a line number inside of jQuery, this does not mean the bug is a jQuery bug; more often than not, these errors result from providing incorrect arguments when calling a jQuery function.

If you are new to jQuery, it is usually a much better idea to ask for help first in the Using jQuery Forum or the jQuery IRC channel. You will get much quicker support, and you will help avoid tying up the jQuery team with invalid bug reports. These same resources can also be useful if you want to confirm that your bug is indeed a bug in jQuery before filing any tickets.

Disable any browser extensions

Make sure you have reproduced the bug with all browser extensions and add-ons disabled, as these can sometimes cause things to break in interesting and unpredictable ways. Try using incognito, stealth or anonymous browsing modes.

Try the latest version of jQuery

Bugs in old versions of jQuery may have already been fixed. In order to avoid reporting known issues, make sure you are always testing against the latest build.

Try an older version of jQuery

Sometimes, bugs are introduced in newer versions of jQuery that do not exist in previous versions. When possible, it can be useful to try testing with an older release.

Reduce, reduce, reduce!

When you are experiencing a problem, the most useful thing you can possibly do is to reduce your code to the bare minimum required to reproduce the issue. This makes it much easier to isolate and fix the offending code. Bugs that are reported without reduced test cases take on average 9001% longer to fix than bugs that are submitted with them, so you really should try to do this if at all possible.

jQuery Core Style Guide

See: jQuery Core Style Guide

Tips For Bug Patching

Environment: localhost

To test the plugin you will need:

  • Some kind of localhost server(any will do)
  • Node.js
  • NPM (comes with the latest version of Node.js)
  • Grunt (install with: npm install grunt -g)

Build a Local Copy of the plugin

Create a fork of the plugin repo on github at http://github.com/jquery/jquery-migrate

Change directory to your web root directory, whatever that might be:

$ cd /path/to/your/www/root/

Clone your plugin fork to work locally

$ git clone git@github.com:username/jquery-migrate.git

Change directory to the newly created dir jquery-migrate/

$ cd jquery-migrate

Add the jquery-migrate master as a remote. I label mine "upstream"

$ git remote add upstream git://github.com/jquery/jquery-migrate.git

Get in the habit of pulling in the "upstream" master to stay up to date as jQuery receives new commits

$ git pull upstream master

Run the Grunt tools:

$ grunt && grunt watch

Now open the jQuery test suite in a browser at http://localhost/test. If there is a port, be sure to include it.

Success! You just built and tested jQuery!

Fix a bug from a ticket filed at bugs.jquery.com:

NEVER write your patches to the master branch

ALWAYS USE A "TOPIC" BRANCH! Like so (#### = the ticket #)...

Make sure you start with your up-to-date master:

$ git checkout master

Create and checkout a new branch that includes the ticket #

$ git checkout -b bug_####

# ( Explanation: this useful command will:
# "checkout" a "-b" (branch) by the name of "bug_####"
# or create it if it doesn't exist )

Now you're on branch: bug_####

Determine the module/file you'll be working in...

Open up the corresponding /test/unit/?????.js and add the initial failing unit tests. This may seem awkward at first, but in the long run it will make sense. To truly and efficiently patch a bug, you need to be working against that bug.

Next, open the module files and make your changes

Run http://localhost/test --> ALL TESTS MUST PASS

Once you're satisfied with your patch...

Stage the files to be tracked:

$ git add filename
# (you can use "git status" to list the files you've changed)

( I recommend NEVER, EVER using "git add . " )

Once you've staged all of your changed files, go ahead and commit them

$ git commit -m "Brief description of fix. Fixes #0000"

For a multiple line commit message, leave off the -m "description".

You will then be led into vi (or the text editor that you have set up) to complete your commit message.

Then, push your branch with the bug fix commits to your github fork

$ git push origin -u bug_####

Before you tackle your next bug patch, return to the master:

$ git checkout master

Test Suite Tips...

By default the plugin runs against the current (jquery-git WIP) version of jQuery. You can select a different version by specifying it in the URL. Files are always retrieved from code.jquery.com.

Example:

http://localhost/jquery-migrate/test/?jquery=1.7.2

This will run the plugin with version 1.7.2 of jQuery, taken from http://code.jquery.com/jquery-1.7.2.js.

ALWAYS RUN THE FULL TEST SUITE BEFORE COMMITTING AND PUSHING A PATCH!