Skip to content

Allow leading underscore in sketch filenames #510

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 2 commits into from
Mar 13, 2023
Merged

Allow leading underscore in sketch filenames #510

merged 2 commits into from
Mar 13, 2023

Conversation

per1234
Copy link
Contributor

@per1234 per1234 commented Mar 13, 2023

The Arduino Sketch Specification defines the allowed format of sketch folder names and sketch code filenames. Arduino Lint's rule SS002 checks compliance with the specification.

The Arduino Sketch Specification is being changed to allow a leading underscore in sketch folder names and sketch code filenames:

arduino/arduino-cli#2105

The Arduino Lint rule code must be updated accordingly.

per1234 added 2 commits March 12, 2023 19:57
Previously the test data for the unit tests of this function did not provide full coverage of all valid sketch filename
formats.
The Arduino Sketch Specification defines the allowed format of sketch folder names and sketch code filenames. Arduino
Lint's rule SS002 checks compliance with the specification.

The Arduino Sketch Specification has been changed to allow a leading underscore in sketch folder names and sketch code
filenames so the Arduino Lint rule code must be updated accordingly.
@per1234 per1234 added topic: code Related to content of the project itself type: imperfection Perceived defect in any part of project labels Mar 13, 2023
@per1234 per1234 self-assigned this Mar 13, 2023
@per1234 per1234 changed the title Allow sketch leading underscore Allow leading underscore in sketch filenames Mar 13, 2023
@codecov-commenter
Copy link

codecov-commenter commented Mar 13, 2023

Codecov Report

Patch and project coverage have no change.

Comparison is base (059c3a0) 90.05% compared to head (fa2d53e) 90.05%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #510   +/-   ##
=======================================
  Coverage   90.05%   90.05%           
=======================================
  Files          44       44           
  Lines        6800     6800           
=======================================
  Hits         6124     6124           
  Misses        553      553           
  Partials      123      123           
Flag Coverage Δ
unit 90.05% <ø> (ø)

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

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@per1234 per1234 merged commit c2c82d6 into arduino:main Mar 13, 2023
@per1234 per1234 deleted the allow_sketch_leading_underscore branch May 12, 2023 04:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
topic: code Related to content of the project itself type: imperfection Perceived defect in any part of project
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants