Skip to content

[skip changelog] Update Platform Specification link in warning #1199

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 1 commit into from
Feb 26, 2021
Merged

[skip changelog] Update Platform Specification link in warning #1199

merged 1 commit into from
Feb 26, 2021

Conversation

per1234
Copy link
Contributor

@per1234 per1234 commented Feb 26, 2021

Please check if the PR fulfills these requirements

  • The PR has no duplicates (please search among the Pull Requests
    before creating one)
  • The PR follows
    our contributing guidelines
  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been added / updated (for bug fixes / features)
  • UPGRADING.md has been updated with a migration guide (for breaking changes)
  • What kind of change does this PR introduce?

docs update

  • What is the current behavior?

When a boards platform uses an outdated recipe.ar.pattern, Arduino CLI displays a helpful message:

Unable to cache built core, please tell foo:[email protected] maintainers to follow http://goo.gl/QdCUjo

That short link points to the old Arduino Platform Specification page in the arduino/Arduino repository's wiki. The platform specification has since been moved to the arduino/arduino-cli repository.

Even though the old page provides a link to the new location, this message uses an anchor to direct the developer to the relevant section of the specification. When they follow the link from the old page, they will simply end up at the top of the specification and may have a hard time finding the information that tells them how to fix the platform.

  • What is the new behavior?

The link is updated to point directly to the relevant information:
https://arduino.github.io/arduino-cli/latest/platform-specification/#recipes-to-build-the-corea-archive-file

Does this PR introduce a breaking change

No

is titled accordingly

Yes

  • Other information:

Perhaps it would be better to create a new short link or update the old one, but I think this fix is fine. This message will not be encountered often because it has been a long time since that part of the platform system was changed.

When a boards platform uses an outdated `recipe.ar.pattern`, Arduino CLI displays a helpful message:

Unable to cache built core, please tell foo:[email protected] maintainers to follow http://goo.gl/QdCUjo

That short link points to the old Arduino Platform Specification page in the arduino/Arduino repository's wiki. The
platform specification has since been moved to the arduino/arduino-cli repository.

Even though the old page provides a
link to the new location, this message uses an anchor to direct the developer to the relevant section of the
specification. When they follow the link from the old page, they will simply end up at the top of the specification and
may have a hard time finding the information that tells them how to fix the platform. Updating the link restores the
intended behavior.
Copy link

@ubidefeo ubidefeo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@per1234 per1234 merged commit 3324298 into arduino:master Feb 26, 2021
@per1234 per1234 deleted the update-warning-link branch February 26, 2021 09:04
@per1234 per1234 self-assigned this Nov 23, 2021
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.

2 participants