Skip to content

Developer Guide: Minor fixes (broken link, missing internal link, extraneous/trailing whitespace) - #12384

Merged
michaelDCurran merged 1 commit into
nvaccess:masterfrom
accessolutions:developerGuideMinorFixes
May 9, 2021
Merged

Developer Guide: Minor fixes (broken link, missing internal link, extraneous/trailing whitespace)#12384
michaelDCurran merged 1 commit into
nvaccess:masterfrom
accessolutions:developerGuideMinorFixes

Conversation

@JulienCochuyt

Copy link
Copy Markdown
Contributor

Link to issue number:

Found while working on updating the Developer Guide for PR #10172.

Summary of the issue:

  • There is a broken link to 7-zip
  • An internal link is missing when referring to the "Packaging code as NVDA Add-ons" section
  • There are a few extraneous or trailing whitespace

Description of how this pull request fixes the issue:

  • Fix broken link to 7-zip
  • Add the missing internal link to the "Packaging code as NVDA Add-ons" section
  • Remove extraneous or trailing whitespace

Testing strategy:

Generated the HTML rendering from source and proof-read the result.

Known issues with pull request:

Change log entries:

These small fixes IMHO do not deserve a change log entry.

Code Review Checklist:

  • Pull Request description is up to date.
  • Unit tests.
  • System (end to end) tests.
  • Manual tests.
  • User Documentation.
  • Change log entry.
  • Context sensitive help for GUI changes.

 * Fix broken link to 7-zip
 * Add a missing internal link to the "Packaging code as NVDA Add-ons" section
 * Remove extraneous or trailing whitespace
@JulienCochuyt
JulienCochuyt requested a review from a team as a code owner May 9, 2021 08:15
@michaelDCurran
michaelDCurran merged commit f8c1826 into nvaccess:master May 9, 2021
@nvaccessAuto nvaccessAuto added this to the 2021.1 milestone May 9, 2021
@JulienCochuyt
JulienCochuyt deleted the developerGuideMinorFixes branch May 9, 2021 23:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants