Skip to content

Improve and correct links in documentation #351

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 5 commits into from
Mar 15, 2022
Merged

Improve and correct links in documentation #351

merged 5 commits into from
Mar 15, 2022

Conversation

per1234
Copy link
Contributor

@per1234 per1234 commented Mar 12, 2022

Fixes and enhancements identified through a comprehensive review of all links used in documentation:

  • Fix broken link
  • Update links that were relying on redirects
  • Configure links to allow localization where available
  • Always point to "latest" version of documentation hosted in Arduino CLI website
  • Target links to relevant section of pages

@per1234 per1234 requested review from silvanocerza and umbynos March 12, 2022 18:06
@per1234 per1234 self-assigned this Mar 12, 2022
@codecov-commenter
Copy link

codecov-commenter commented Mar 12, 2022

Codecov Report

Merging #351 (34e5da2) into main (b139ce8) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #351   +/-   ##
=======================================
  Coverage   90.01%   90.01%           
=======================================
  Files          44       44           
  Lines        6770     6770           
=======================================
  Hits         6094     6094           
  Misses        553      553           
  Partials      123      123           
Flag Coverage Δ
unit 90.01% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
...ternal/rule/ruleconfiguration/ruleconfiguration.go 100.00% <ø> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b5d7bb8...34e5da2. Read the comment docs.

per1234 added 5 commits March 15, 2022 03:11
The Arduino Library Manager documentation was moved to a new location without setting up a redirect, which broke this link.
Even though these links were still reaching the intended page, it is best not to rely on redirects.
github.com and wikipedia.org have different localization behaviors depending on the URL.

If a language code is specified via the URL, then that language version of the page is loaded, regardless of the language setting of the user's browser. For example, this URL will take the user to the English version of the page even if their browser is configured for Italian:

https://en.wikipedia.org/wiki/Arduino

If no language code is specified via the URL, then it redirects to the version of the page localized for the user's language preference, where available. For example, if the user has selected "Italian" as their preferred language in their browser settings, then this URL:

https://wikipedia.org/wiki/Arduino

redirects to:

https://it.wikipedia.org/wiki/Arduino
The documentation hosted at arduino.github.io/arduino-cli is versioned, with the URL determining which version is loaded:

- arduino.github.io/arduino-cli/0.20 - documentation content at the time of the latest release of the 0.20.x series
- arduino.github.io/arduino-cli/latest - documentation content at the time of the latest release
- arduino.github.io/arduino-cli/dev - documentation content at the tip of the default branch

Previously, some of the links displayed to users by Arduino Lint were pointing at the "dev" version of that
documentation. Since it may not apply to the release versions, or be in an unstable state, the "dev" version of the
documentation should only be used by Arduino CLI developers and beta testers. For this reason, the "latest" version will
be most appropriate for the users of Arduino Lint.
The Arduino Sketch Specification document has had some recent changes to avoid ambiguity. This allows the reference
links to now point to the specific section of the specification relevant to that rule so that the user will be able to
find the information quickly.
@per1234 per1234 added topic: documentation Related to documentation for the project type: enhancement Proposed improvement type: imperfection Perceived defect in any part of project labels Mar 15, 2022
@per1234 per1234 merged commit 4f0fa8b into arduino:main Mar 15, 2022
@per1234 per1234 deleted the fix-link branch March 15, 2022 10:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
topic: documentation Related to documentation for the project type: enhancement Proposed improvement type: imperfection Perceived defect in any part of project
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants