Skip to content

Conversation

ramya-subramanyam
Copy link
Collaborator

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 ---

@ramya-subramanyam ramya-subramanyam force-pushed the psoc-ino-unity-tests-hil-setup-1 branch from 9d355e0 to 1d5c50e Compare August 4, 2025 05:42
Copy link
Member

@jaenrig-ifx jaenrig-ifx left a comment

Choose a reason for hiding this comment

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

:) Nice :)

@ramya-subramanyam ramya-subramanyam force-pushed the psoc-ino-unity-tests-hil-setup-1 branch from f37e203 to cedc5a3 Compare August 6, 2025 08:51
@ramya-subramanyam ramya-subramanyam force-pushed the psoc-ino-unity-tests-hil-setup-1 branch from cedc5a3 to 1161c2f Compare August 6, 2025 11:57
@ramya-subramanyam ramya-subramanyam force-pushed the psoc-ino-unity-tests-hil-setup-1 branch from 746b914 to ec1bf1e Compare August 7, 2025 05:19
options:
USE_CORE:
name: local
url: http://blabla
Copy link
Member

Choose a reason for hiding this comment

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

http://blabla looks like a placeholder?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Yes. it should be additional-urls. I will check this in the next iteration when I am adding the remaining tests.

Copy link
Member

@jaenrig-ifx jaenrig-ifx left a comment

Choose a reason for hiding this comment

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

All good. The only thing:
Why "arduino-devops" is required as submodule?

@ramya-subramanyam ramya-subramanyam force-pushed the psoc-ino-unity-tests-hil-setup-1 branch from 7dd80eb to a907142 Compare August 7, 2025 07:37
@ramya-subramanyam ramya-subramanyam force-pushed the psoc-ino-unity-tests-hil-setup-1 branch from a907142 to dbf1a1a Compare August 7, 2025 07:44
@ramya-subramanyam
Copy link
Collaborator Author

All good. The only thing: Why "arduino-devops" is required as submodule?

I hadnt added it till today morning after realizing this is why all the tests were failing from yesterday.
The issue is path extras/arduino-devops is being used in multiple makers-devops script to locally instal the core packages. We need to check this again and rework on the scripts.

@ramya-subramanyam ramya-subramanyam merged commit a576249 into main Aug 7, 2025
22 checks passed
@ramya-subramanyam ramya-subramanyam deleted the psoc-ino-unity-tests-hil-setup-1 branch August 7, 2025 07:56
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.

3 participants