Skip to content

Conversation

jaenrig-ifx
Copy link
Member

@jaenrig-ifx jaenrig-ifx commented Jul 18, 2025

By creating this pull request you agree to the terms in CONTRIBUTING.md.
https://github.com/Infineon/.github/blob/master/CONTRIBUTING.md
--- DO NOT DELETE ANYTHING ABOVE THIS LINE ---

CONTRIBUTING.md also tells you what to expect in the PR process.

  • Added missing integer to char functions (not provided by the standard library). The file is taken from the renesas Arduino core.
  • Added String() constructor examples.

@ederjc
Copy link
Member

ederjc commented Jul 18, 2025

the license will not be detected by GitHub like this. Can we find a different solution? Second license file in the respective folder?

@jaenrig-ifx
Copy link
Member Author

I still need to test on target. If that works I will open it from the draft.
But this is the reason why this is not compiling:
https://github.com/Infineon/arduino-radar-bgt60/actions/runs/16027236523/job/45218247412?pr=5
And it is not by default provided by Arduino-CoreAPI:
arduino/ArduinoCore-nRF528x-mbedos#6

@jaenrig-ifx
Copy link
Member Author

we find a different solution? Second license file in the respective folder?

Anything below the LICENSE will make GitHub not determine automatically the LICENSE you mean?
Let´s research a bit how, I keep that out of this PR. I would prefer not to maintain bunch of LICENSE files per directory.

@jaenrig-ifx jaenrig-ifx marked this pull request as ready for review July 21, 2025 07:44
@jaenrig-ifx jaenrig-ifx force-pushed the fix/add-missing-itoa-ultoa branch from 00fe26e to 696238c Compare July 21, 2025 07:45
@jaenrig-ifx jaenrig-ifx merged commit 5835ca4 into main Aug 4, 2025
11 checks passed
@jaenrig-ifx jaenrig-ifx deleted the fix/add-missing-itoa-ultoa branch August 4, 2025 08:00
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.

4 participants