Skip to content

Latest commit

 

History

History
151 lines (120 loc) · 7.36 KB

CONTRIBUTING.md

File metadata and controls

151 lines (120 loc) · 7.36 KB
Raw

Contributing to libphonenumber

Thanks for contributing to libphonenumber!

Please read the following before filing an issue or sending a pull request.

We hope these guidelines will enhance your experience as a contributor to our library and know that we appreciate the time you put into making it better.

Checklist before filing an issue

Please check the following:

  • Is the issue reproducible using the demo? If not:
    • Your issue may be resolved by upgrading to the latest version of the library. (Reach out to the product/app team if you're not the developer!)
    • If you are using a port, issues should be reported against that project.
    • If you are using the library through a front-end or wrapper, issues should be reported against that project.
  • Has the issue already been discussed in the issues or discussion group? If so, you may want to comment on an existing discussion.

Filing a metadata issue

Please copy this template into your report and answer the questions to the best of your ability. For acceptable evidence, see below.

*   Country:
*   Example number(s) and/or range(s):
*   Number type ("fixed-line", "mobile", "short code", etc.):
*   For short codes, cost and dialing restrictions:
*   Where or whom did you get the number(s) from:
*   Authoritative evidence (e.g. national numbering plan, operator announcement):
*   Link from demo (http://libphonenumber.appspot.com) showing error:

From mid December to early January, we kindly request that you use Google's Issue Tracker to report issues with the library. During this period we'll turn off the "Issues" tab here on GitHub. We apologize for any inconvenience (e.g. accessing old report) while we test this system!

Alternatively, if you have a particular issue with a Google product handling phone numbers (e.g. signup), you may also use the Google Product Forums.

Provide authoritative evidence

Ideally, change requests for ranges should be accompanied by authoritative evidence such as official government or public carrier documents.

If the evidence is publicly available online, please provide the link.

If the evidence is not publicly available online, make sure that you have the rights to share this with us, and confirm this by reading and signing the appropriate Contributor License Agreement (CLA).

See https://cla.developers.google.com/about to determine whether you need to sign a Corporate or Individual CLA.

By signing the CLA, you confirm that you have the rights to share the information with us, and that we may use, modify, reproduce, publicly display, and distribute the information in accordance with the library's open source license.

  • Corporate CLA:

    • See https://cla.developers.google.com/about and follow the instructions.
    • Once you are all set up, either create a validation issue for your request, or go to the relevant open issue.
    • Comment on the issue, using the GitHub account that's already set up for your Corporate CLA, with "I signed the CLA."
    • If you're set up correctly, someone from the team will update you on the issue, with instructions for sending private documentation to our closed mailing list. Only the core libphonenumber team has access to view emails sent to this address.

  • Individual CLA:

    • See https://cla.developers.google.com/about to ensure that an Individual CLA is what you need to sign, and for help with troubleshooting.
    • When signing the Individual CLA, use the same GitHub username you used (or will use) to file the corresponding issue.
    • If the email address you use in the Individual CLA is not already associated with that GitHub account, add it.
    • If you need to modify an Individual CLA signature, such as to change the GitHub username, go to https://cla.developers.google.com/clas and click on "Edit Contact Information".
    • Once you are all set up, either create a validation issue for your request, or go to the relevant open issue.
    • Comment on the issue, using the GitHub account that's already set up for your Individual CLA, with "I signed the CLA."
    • If you're set up correctly, someone from the team will update you on the issue, with instructions for sending private documentation to our closed mailing list. Only the core libphonenumber team has access to view emails sent to this address.

Pull requests

We do not accept pull requests for validation, formatting, or timezone metadata updates.

For changes specific to windows builds, see Filing a code issue and make sure you have found a reviewer and tester before sending the pull request.

We are happy to review and accept pull requests for the following:

  • Carrier updates
  • Geocoding updates
  • Bug fixes
  • Documentation and code hygiene cleanups

This list is not exhaustive. To clarify whether we'd accept a pull request, and especially before spending significant time on one such as for a bug fix, we strongly encourage that you bring up the question on an issue.

To get your pull request merged, we need the following:

  • A CLA signature. See below.
  • Responsiveness to back and forth review comments, because we will do a code review.
  • A link to a clear description of what the pull request intends to solve in a GitHub issue or Google's Issue Tracker.
  • Tests that illustrate the issue, if applicable.
  • Willingness and ability to implement the changes in C++, Java, and JS if applicable. For example, changes to PhoneNumberUtil.java must be ported to phonenumberutil.cc and phonenumberutil.js.
    • The initial pull request may contain the implementation in only one language to get approval on the approach.

If this is not possible, please file an issue instead.

Filing a code issue

When filing a code issue, include the specifics of your operating system and provide as much information as possible that helps us reproduce the problem.

Please be advised that metadata updates are prioritized over code changes, except for bug fixes. In addition, we work with a limited number of build systems and may not be able to support every setup.

In particular, we don't actively maintain windows builds but would be able to accept a PR provided someone else in the open-source community could test it out and would be able to help with the review. One way to look for such collaborators would be to email the discussion group. Also see the known Windows issues.