|
123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185 |
- Contributing to Leaflet
- =======================
-
- 1. [Getting Involved](#getting-involved)
- 2. [Reporting Bugs](#reporting-bugs)
- 3. [Contributing Code](#contributing-code)
- 4. [Improving Documentation](#improving-documentation)
- 5. [Code of Conduct](#code-of-conduct)
-
- ## Getting Involved
-
- Third-party patches are absolutely essential on our quest to create the best mapping library that will ever exist.
- However, they're not the only way to get involved with Leaflet development.
- You can help the project tremendously by discovering and [reporting bugs](#reporting-bugs);
- [improving documentation](#improving-documentation);
- helping others on [Stack Overflow](https://stackoverflow.com/questions/tagged/leaflet),
- [GIS Stack Exchange](https://gis.stackexchange.com/questions/tagged/leaflet)
- and [GitHub issues](https://github.com/Leaflet/Leaflet/issues);
- showing your support for your favorite feature suggestions on [Leaflet UserVoice page](http://leaflet.uservoice.com);
- tweeting to [@LeafletJS](http://twitter.com/LeafletJS);
- and spreading the word about Leaflet among your colleagues and friends.
-
- ## Reporting Bugs
-
- Before reporting a bug on the project's [issues page](https://github.com/Leaflet/Leaflet/issues),
- first make sure that your issue is caused by Leaflet, not your application code
- (e.g. passing incorrect arguments to methods, etc.).
- Second, search the already reported issues for similar cases,
- and if it's already reported, just add any additional details in the comments.
-
- After you've made sure that you've found a new Leaflet bug,
- here are some tips for creating a helpful report that will make fixing it much easier and quicker:
-
- * Write a **descriptive, specific title**. Bad: *Problem with polylines*. Good: *Doing X in IE9 causes Z*.
- * Include **browser, OS and Leaflet version** info in the description.
- * Create a **simple test case** that demonstrates the bug (e.g. using [Leaflet playground](http://playground-leaflet.rhcloud.com/)).
- * Check whether the bug can be reproduced in **other browsers**.
- * Check if the bug occurs in the stable version, master, or both.
- * *Bonus tip:* if the bug only appears in the master version but the stable version is fine,
- use `git bisect` to find the exact commit that introduced the bug.
-
- If you just want some help with your project,
- try asking on [Stack Overflow](https://stackoverflow.com/questions/tagged/leaflet)
- or [GIS Stack Exchange](https://gis.stackexchange.com/questions/tagged/leaflet) instead.
-
- ## Contributing Code
-
- ### Considerations for Accepting Patches
-
- While we happily accept patches, we're also committed to keeping Leaflet simple, lightweight and blazingly fast.
- So bugfixes, performance optimizations and small improvements that don't add a lot of code
- are much more likely to get accepted quickly.
-
- Before sending a pull request with a new feature, check if it's been discussed before already
- (either on [GitHub issues](https://github.com/Leaflet/Leaflet/issues)
- or [Leaflet UserVoice](http://leaflet.uservoice.com/)),
- and ask yourself two questions:
-
- 1. Are you sure that this new feature is important enough to justify its presence in the Leaflet core?
- Or will it look better as a plugin in a separate repository?
- 2. Is it written in a simple, concise way that doesn't add bulk to the codebase?
-
- If your feature or API improvement did get merged into master,
- please consider submitting another pull request with the corresponding [documentation update](#improving-documentation).
-
- ### Setting up the Build System
-
- The Leaflet build system uses [Node](http://nodejs.org/), and the [Jake](http://jakejs.com/) Javascript build tool.
- To set up the Leaflet build system, install Node then run the following commands in the project root to install Jake:
-
- ```
- npm install -g jake
- npm install
- ```
-
- You can build minified Leaflet by running `jake` (it will be built from source in the `dist` folder).
- For a custom build with selected components, open `build/build.html` in the browser and follow the instructions from there.
-
- ### Making Changes to Leaflet Source
-
- If you're not yet familiar with the way GitHub works (forking, pull requests, etc.),
- be sure to check out the awesome [article about forking](https://help.github.com/articles/fork-a-repo)
- on the GitHub Help website — it will get you started quickly.
-
- You should always write each batch of changes (feature, bugfix, etc.) in **its own topic branch**.
- Please do not commit to the `master` branch, or your unrelated changes will go into the same pull request.
-
- You should also follow the code style and whitespace conventions of the original codebase.
- In particular, use tabs for indentation and spaces for alignment.
-
- Before committing your changes, run `jake lint` to catch any JS errors in the code and fix them.
- If you add any new files to the Leaflet source, make sure to also add them to `build/deps.js`
- so that the build system knows about them.
-
- Also, please make sure that you have [line endings configured properly](https://help.github.com/articles/dealing-with-line-endings) in Git! Otherwise the diff will show that all lines of a file were changed even if you touched only one.
-
- Happy coding!
-
- ## Running the Tests
-
- To run the tests from the command line,
- install [PhantomJS](http://phantomjs.org/) (and make sure it's in your `PATH`),
- then run:
-
- ```
- jake test
- ```
-
- To run all the tests in actual browsers at the same time, you can do:
-
- ```
- jake test --ff --chrome --safari --ie
- ```
-
- To run the tests in a browser manually, open `spec/index.html`.
-
- ## Code Coverage
-
- To generate a detailed report about test coverage (which helps tremendously when working on test improvements), run:
-
- ```
- jake test --cov
- ```
-
- After that, open `coverage/<environment>/index.html` in a browser to see the report.
- From there you can click through folders/files to get details on their individual coverage.
-
- ## Improving Documentation
-
- The code of the live Leaflet website that contains all documentation and examples is located in the `docs/` directory of the `master` branch
- and is automatically generated from a set of HTML and Markdown files by [Jekyll](http://jekyllrb.com/).
-
- The easiest way to make little improvements such as fixing typos without even leaving the browser
- is by editing one of the files with the online GitHub editor:
- browse the [`docs/ directory`](https://github.com/Leaflet/Leaflet/tree/master/docs),
- choose a certain file for editing (e.g. `plugins.html` for the list of Leaflet plugins),
- click the Edit button, make changes and follow instructions from there.
- Once it gets merged, the changes will immediately appear on the website.
-
- If you need to make edits in a local repository to see how it looks in the process, do the following:
-
- 1. [Install Ruby](http://www.ruby-lang.org/en/) if don't have it yet.
- 2. Run `gem install jekyll`.
- 3. Enter the directory where you cloned the Leaflet repository
- 4. Make sure you are in the `master` branch by running `git checkout master`
- 5. Enter the documentation subdirectory by running `cd docs`
- 6. Run `jekyll serve --watch`.
- 7. Open `localhost:4000` in your web browser.
-
- Now any file changes will be updated when you reload pages automatically.
- After committing the changes, just send a pull request.
-
- ### API documentation
-
- Since Leaflet 1.0.0-rc1, the API documentation in `reference-1.0.0.html` is handled
- via [Leafdoc](https://github.com/Leaflet/Leafdoc). This means that next to the
- code for every method, option or property there is a special code comment documenting
- that feature. In order to edit the API documentation, just edit these comments in the
- source code.
-
- In order to generate the documentation, just run
-
- ```
- jake docs
- ```
-
- and you'll find a `.html` file in the `dist/` directory.
-
- On every release of a new Leaflet version, this file will be generated and copied
- over to `docs/reference.html` - there is no need to send pull requests with changes to this file to update the API documentation.
-
- ## Code of Conduct
-
- Everyone is invited to participate in the Leaflet community and related projects:
- we want to create a welcoming and friendly environment.
- Harassment of participants or other unethical and unprofessional behavior will not be tolerated in our spaces.
- The [Contributor Covenant](http://contributor-covenant.org/version/1/3/0/)
- applies to all projects under the Leaflet organization.
- Report any issues to agafonkin@gmail.com.
-
- ## Thank You
-
- Not only does your contribution to Leaflet and its community earn our gratitude, but it also makes you AWESOME.
- Join [this approved list of awesome people](https://github.com/Leaflet/Leaflet/graphs/contributors)
- and help us push the limits of what's possible with online maps!
|