Skip to content

Latest commit

 

History

History
95 lines (82 loc) · 6.08 KB

contributing.md

File metadata and controls

95 lines (82 loc) · 6.08 KB

Code/Docs contributions

If you are new to the project, or looking for some way to help take a look at:

Guidelines:

  • For anything other than minor bug fixes, please comment on an existing issue or create a new issue providing details about your proposed change.
  • Unrelated changes should be addressed in separate issues.
  • Include information about use cases, design, user experience, etc.
    • This allows us to discuss these aspects and any other concerns that might arise, thus potentially avoiding a great deal of wasted time.
  • It is recommended to wait for acceptance of your proposal before you start coding.
    • A triaged label is an indicator that an issue is ready for a fix.
    • Please understand that we very likely will not accept changes that are not discussed first.
    • Consider starting a GitHub discussion or mailing list topic to see if there is interest.
  • A minor/trivial change which definitely wouldn't require design, user experience or implementation discussion, you can just create a pull request rather than using an issue first.
    • e.g. a fix for a typo/obvious coding error or a simple synthesizer/braille display driver
    • This should be fairly rare.
  • If in doubt, use an issue first. Use this issue to discuss the alternatives you have considered in regards to implementation, design, and user experience. Then give people time to offer feedback.

Overview of contribution process:

  1. Setup your development environment.
  2. Ensure the issue you plan to fix is triaged
  3. Create a branch for the contribution, to be used for a pull request.
    • Pull requests should be based on the latest commit in the official master branch. This helps reduce the chance of merge conflicts.
    • If you are adding a feature or changing something that will be noticeable to the user, you should update the User Guide accordingly. New commands, drivers, settings, dialogs, etc. must be documented.
  4. Build NVDA and run from source
  5. Manually test the change
  6. Run automated tests
    • Run rununittests (rununittests.bat) before you open your Pull Request, and make sure all the unit tests pass.
    • If possible for your PR, please consider creating a set of unit or system tests to test your changes.
    • The lint check ensures your changes comply with our code style expectations. Use runlint nvaccess/master (runlint.bat)
    • Run scons checkPot to ensure translatable strings have comments for the translators
  7. Create a change log entry
  8. Create a Pull Request (PR)
    • When you think a contribution is ready, or you would like feedback, open a draft pull request. When you would like a review, mark the PR as "ready for review".
    • Please fill out the Pull Request template, including the checklist of considerations. The checklist asks you to confirm that you have thought about each of the items, if any of the items are missing it is helpful to explain elsewhere in the PR why it has been left out.
    • Automated checks will be run against your PR. If these fail, please review them. Sometimes system tests fail unexpectedly. If you believe the failure is unrelated, feel free to ignore it unless it is raised by a reviewer.
    • All pull requests submitted must have their "Allow edits from maintainers" checkbox ticked. This is the GitHub default for new pull requests.
    • A lead developer may specifically request the pull request be made against beta or rc in the case of addressing bugs introduced in the current release cycle.
  9. Participate in the code review process
    • This process requires core NVDA developers to understand the intent of the change, read the code changes, asking questions or suggesting changes. Please participate in this process, answering questions, and discussing the changes.
    • Being proactive will really help to speed up the process of code review.
    • When the PR is approved it will be merged, and the change will be active in the next alpha build.
    • If issues are raised with your PR, it may be marked as a draft. Please mark it as ready for review when you have addressed the review comments.
  10. Feedback from alpha users
    • After a PR is merged, watch for feedback from alpha users / testers. You may have to follow up to address bugs or missed use-cases.

Change log entry

An entry intended to explain changes in NVDA to end users. Your proposed entry should be added to the changes.t2t file which is converted to HTML. Change log entries are not required for changes with no/minor user impact or no developer impact.

Because the changes.t2t file is prone to conflicts, NV Access will resolve any merge conflicts with the change log entry before merging.

These descriptions should be in the format: "{Description of change}. (#{issue number})". Multiple issue numbers can be included, separated by comma. If there is no issue number, you can use the PR number.

For instance:

New features
- Added a command to announce useful thing. (#WXYZ, #ABCD)

Changes
- Old command, now also uses new useful command. (#WXYZ)

You may add descriptions for multiple sections. The sections are:

  • New features
  • Changes
  • Bug fixes
  • Changes for developers

Code Style

Please ensure you follow our coding standards document.

Technical design

Please checkout our technical design overview.