Skip to content

[skip changelog] Correct library+sketch specifications re: spaces in folder name #1194

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] Correct library+sketch specifications re: spaces in folder name #1194

merged 1 commit into from
Feb 26, 2021

Conversation

per1234
Copy link
Contributor

@per1234 per1234 commented Feb 24, 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?

The library and sketch specifications say that spaces are allowed in library and sketch folder names, but this is not supported by the Arduino IDE. Use of spaces in the folder name will result in a warning on IDE startup and the project not being recognized.

The previous mention of spaces as one of the characters allowed in library and sketch folder names was a copy/paste
error originating in the allowed characters in library.properties name properties. This specific part of the name
property specification doesn't apply to folder names. The reason it is allowed in the name property is because that
value is "sanitized" before being used by Library Manager for the library installation folder name by replacing all
spaces with underscores.

  • What is the new behavior?
    The erroneous mention of spaces as one of the allowed characters for library and sketch folder names is removed.

No

  • Other information:

It was previously necessary to disable Prettier formatting of the sentence in question in the sketch specification:

<!-- prettier-ignore -->

The reason for this was that prettier removed the space from between the backticks in that sentence. Now that the space has been removed from the sentence, it can be safely formatted by Prettier.


See how to contribute

…folder name

The previous mention of spaces as one of the characters allowed in library and sketch folder names was a copy/paste
error originating in the allowed characters in library.properties `name` properties. This specific part of the `name`
property specification doesn't apply to folder names. The reason it is allowed in the `name` property is because that
value is "sanitized" before being used by Library Manager for the library installation folder name by replacing all
spaces with underscores.
@per1234 per1234 changed the title Correct library+sketch specifications re: spaces in folder name [skip changelog] Correct library+sketch specifications re: spaces in folder name Feb 24, 2021
@per1234 per1234 requested a review from silvanocerza February 26, 2021 05:50
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 3306750 into arduino:master Feb 26, 2021
@per1234 per1234 deleted the correct-sketch-library-folder-name-spec 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