Skip to content

Commit e0c25ab

Browse files
committed
Docs: add a note about loading source with AMD
- Moves the note about the watch task and the note about loading with AMD to their own section under "Test Suite Tips" Fixes gh-2714 Close gh-2725
1 parent cb80b42 commit e0c25ab

File tree

1 file changed

+14
-7
lines changed

1 file changed

+14
-7
lines changed

CONTRIBUTING.md

Lines changed: 14 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -101,12 +101,6 @@ Run the build script
101101
$ npm run build
102102
```
103103

104-
Run the Grunt tools:
105-
106-
```bash
107-
$ grunt && grunt watch
108-
```
109-
110104
Now open the jQuery test suite in a browser at http://localhost/test. If there is a port, be sure to include it.
111105

112106
Success! You just built and tested jQuery!
@@ -118,13 +112,26 @@ During the process of writing your patch, you will run the test suite MANY times
118112

119113
Example:
120114

121-
http://localhost/test/?filter=css
115+
http://localhost/test/?module=css
122116

123117
This will only run the "css" module tests. This will significantly speed up your development and debugging.
124118

125119
**ALWAYS RUN THE FULL SUITE BEFORE COMMITTING AND PUSHING A PATCH!**
126120

127121

122+
#### Loading changes on the test page
123+
124+
Rather than rebuilding jQuery with `grunt` every time you make a change, you can use the included `grunt watch` task to rebuild distribution files whenever a file is saved.
125+
126+
```bash
127+
$ grunt watch
128+
```
129+
130+
Alternatively, you can **load tests in AMD** to avoid the need for rebuilding altogether.
131+
132+
Click "Load with AMD" after loading the test page.
133+
134+
128135
### Browser support
129136

130137
Remember that jQuery supports multiple browsers and their versions; any contributed code must work in all of them. You can refer to the [browser support page](http://jquery.com/browser-support/) for the current list of supported browsers.

0 commit comments

Comments
 (0)