Skip to content

Conversation

maarten-pennings
Copy link
Contributor

No description provided.

Copy link
Contributor

github-actions bot commented Sep 5, 2024

Hi @maarten-pennings.
Your pull request has been detected as something other than a Library Manager submission.
A maintainer will need to review it before it can be merged.

If you intended to submit a library, please check the instructions and update your pull request if necessary:
https://github.com/arduino/library-registry/blob/main/README.md#instructions

@github-actions github-actions bot added the topic: modification Change existing list entry label Sep 5, 2024
Copy link
Contributor

github-actions bot commented Sep 5, 2024

ERROR: Arduino Lint found errors with https://github.com/ams-OSRAM-Group/OSP_aoui32:

Linting library in OSP_aoui32
INFO: No header file found matching library name (OSP_UIDriversOSP32_aoui32.h). Best practices are for primary header   
      filename to match library name.                                                                                   
      See: https://arduino.github.io/arduino-cli/latest/library-specification/#libraryproperties-file-format            
      (Rule LS008)                                                                                                      
WARNING: library.properties name value OSP UIDriversOSP32 aoui32 is longer than the recommended length of 16 characters.
         See: https://arduino.github.io/arduino-cli/latest/library-specification/#libraryproperties-file-format         
         (Rule LP010)                                                                                                   
WARNING: library.properties name OSP UIDriversOSP32 aoui32 contains spaces. Although supported, best practices is to not
         use spaces.                                                                                                    
         See: https://arduino.github.io/arduino-cli/latest/library-specification/#libraryproperties-file-format         
         (Rule LP015)                                                                                                   
ERROR: Library name OSP UIDriversOSP32 aoui32 not found in the Library Manager index. Library names are not allowed to  
       change after being added to the index.                                                                           
       See: https://github.com/arduino/library-registry/blob/main/FAQ.md#how-can-i-change-a-librarys-name               
       (Rule LP018)                                                                                                     

Linter results for project: 1 ERRORS, 2 WARNINGS

-------------------

Linting sketch in OSP_aoui32/examples/aoui32_but

Linter results for project: no errors or warnings

-------------------

Linting sketch in OSP_aoui32/examples/aoui32_led

Linter results for project: no errors or warnings

-------------------

Linting sketch in OSP_aoui32/examples/aoui32_oled

Linter results for project: no errors or warnings

-------------------

Linter results for projects: 1 ERRORS, 2 WARNINGS

Copy link
Contributor

github-actions bot commented Sep 5, 2024

ERROR: Arduino Lint found errors with https://github.com/ams-OSRAM-Group/OSP_aomw:

Linting library in OSP_aomw
INFO: No header file found matching library name (OSP_Middleware_aomw.h). Best practices are for primary header filename
      to match library name.                                                                                            
      See: https://arduino.github.io/arduino-cli/latest/library-specification/#libraryproperties-file-format            
      (Rule LS008)                                                                                                      
WARNING: library.properties name value OSP Middleware aomw is longer than the recommended length of 16 characters.
         See: https://arduino.github.io/arduino-cli/latest/library-specification/#libraryproperties-file-format   
         (Rule LP010)                                                                                             
WARNING: library.properties name OSP Middleware aomw contains spaces. Although supported, best practices is to not use  
         spaces.                                                                                                        
         See: https://arduino.github.io/arduino-cli/latest/library-specification/#libraryproperties-file-format         
         (Rule LP015)                                                                                                   
ERROR: Library name OSP Middleware aomw not found in the Library Manager index. Library names are not allowed to change 
       after being added to the index.                                                                                  
       See: https://github.com/arduino/library-registry/blob/main/FAQ.md#how-can-i-change-a-librarys-name               
       (Rule LP018)                                                                                                     

Linter results for project: 1 ERRORS, 2 WARNINGS

-------------------

Linting sketch in OSP_aomw/examples/aomw_eeprom

Linter results for project: no errors or warnings

-------------------

Linting sketch in OSP_aomw/examples/aomw_flag

Linter results for project: no errors or warnings

-------------------

Linting sketch in OSP_aomw/examples/aomw_iox

Linter results for project: no errors or warnings

-------------------

Linting sketch in OSP_aomw/examples/aomw_min

Linter results for project: no errors or warnings

-------------------

Linting sketch in OSP_aomw/examples/aomw_topodemo

Linter results for project: no errors or warnings

-------------------

Linting sketch in OSP_aomw/examples/aomw_topodump

Linter results for project: no errors or warnings

-------------------

Linting sketch in OSP_aomw/examples/aomw_tscript

Linter results for project: no errors or warnings

-------------------

Linter results for projects: 1 ERRORS, 2 WARNINGS

Copy link
Contributor

github-actions bot commented Sep 5, 2024

A problem was found with your submission https://github.com/ams-OSRAM-Group/OSP_aospi

ERROR: Submission URL is already in the Library Manager index.

Copy link
Contributor

github-actions bot commented Sep 5, 2024

Thanks for your interest in contributing to the Arduino Library Manager index @maarten-pennings
Please resolve the error(s) mentioned in the previous comment.

After resolving the issue, trigger this check again by doing one of the following:

  • Commit the required change to the branch you submitted this pull request from.
  • Comment here, mentioning @ArduinoBot in the comment.

NOTE: It is not necessary to open a new pull request. ❗

More information:
https://github.com/arduino/library-registry/blob/main/README.md#if-the-problem-is-with-the-pull-request

@github-actions github-actions bot requested a review from per1234 September 5, 2024 15:29
@github-actions github-actions bot added topic: submission Add library to the list and removed topic: modification Change existing list entry labels Sep 5, 2024
@github-actions github-actions bot removed the request for review from per1234 September 5, 2024 15:45
Copy link
Contributor

github-actions bot commented Sep 5, 2024

Arduino Lint has suggestions for possible improvements to https://github.com/ams-OSRAM-Group/OSP_aomw:

Linting library in OSP_aomw
INFO: No header file found matching library name (OSP_Middleware_aomw.h). Best practices are for primary header filename
      to match library name.                                                                                            
      See: https://arduino.github.io/arduino-cli/latest/library-specification/#libraryproperties-file-format            
      (Rule LS008)                                                                                                      
WARNING: library.properties name value OSP Middleware aomw is longer than the recommended length of 16 characters.
         See: https://arduino.github.io/arduino-cli/latest/library-specification/#libraryproperties-file-format   
         (Rule LP010)                                                                                             
WARNING: library.properties name OSP Middleware aomw contains spaces. Although supported, best practices is to not use  
         spaces.                                                                                                        
         See: https://arduino.github.io/arduino-cli/latest/library-specification/#libraryproperties-file-format         
         (Rule LP015)                                                                                                   

Linter results for project: 0 ERRORS, 2 WARNINGS

-------------------

Linting sketch in OSP_aomw/examples/aomw_eeprom

Linter results for project: no errors or warnings

-------------------

Linting sketch in OSP_aomw/examples/aomw_flag

Linter results for project: no errors or warnings

-------------------

Linting sketch in OSP_aomw/examples/aomw_iox

Linter results for project: no errors or warnings

-------------------

Linting sketch in OSP_aomw/examples/aomw_min

Linter results for project: no errors or warnings

-------------------

Linting sketch in OSP_aomw/examples/aomw_topodemo

Linter results for project: no errors or warnings

-------------------

Linting sketch in OSP_aomw/examples/aomw_topodump

Linter results for project: no errors or warnings

-------------------

Linting sketch in OSP_aomw/examples/aomw_tscript

Linter results for project: no errors or warnings

-------------------

Linter results for projects: 0 ERRORS, 2 WARNINGS

Copy link
Contributor

github-actions bot commented Sep 5, 2024

Arduino Lint has suggestions for possible improvements to https://github.com/ams-OSRAM-Group/OSP_aoui32:

Linting library in OSP_aoui32
INFO: No header file found matching library name (OSP_UIDriversOSP32_aoui32.h). Best practices are for primary header   
      filename to match library name.                                                                                   
      See: https://arduino.github.io/arduino-cli/latest/library-specification/#libraryproperties-file-format            
      (Rule LS008)                                                                                                      
WARNING: library.properties name value OSP UIDriversOSP32 aoui32 is longer than the recommended length of 16 characters.
         See: https://arduino.github.io/arduino-cli/latest/library-specification/#libraryproperties-file-format         
         (Rule LP010)                                                                                                   
WARNING: library.properties name OSP UIDriversOSP32 aoui32 contains spaces. Although supported, best practices is to not
         use spaces.                                                                                                    
         See: https://arduino.github.io/arduino-cli/latest/library-specification/#libraryproperties-file-format         
         (Rule LP015)                                                                                                   

Linter results for project: 0 ERRORS, 2 WARNINGS

-------------------

Linting sketch in OSP_aoui32/examples/aoui32_but

Linter results for project: no errors or warnings

-------------------

Linting sketch in OSP_aoui32/examples/aoui32_led

Linter results for project: no errors or warnings

-------------------

Linting sketch in OSP_aoui32/examples/aoui32_oled

Linter results for project: no errors or warnings

-------------------

Linter results for projects: 0 ERRORS, 2 WARNINGS

@github-actions github-actions bot merged commit 4f5c2c9 into arduino:main Sep 5, 2024
10 checks passed
Copy link
Contributor

github-actions bot commented Sep 5, 2024

Your submission has now been accepted! Thanks for your contribution to the Arduino Library Manager index.

The library(s) will be available for installation via Library Manager within a day's time.

You can check the logs from the Library Manager indexer for your library(s) here:
http://downloads.arduino.cc/libraries/logs/github.com/ams-OSRAM-Group/OSP_aomw/
http://downloads.arduino.cc/libraries/logs/github.com/ams-OSRAM-Group/OSP_aoui32/

github-actions bot pushed a commit that referenced this pull request Sep 5, 2024
@per1234 per1234 self-assigned this Sep 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

topic: submission Add library to the list

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants