Skip to content
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

Add unit test to cicd #28

Open
wants to merge 8 commits into
base: develop
Choose a base branch
from
Open

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

Add HIL tests to run on our CI/CD.

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.

I see that all this is building in the labpc.... Which one? The one in Muc or in BLR? And it only fails in the flashing or?

@@ -70,22 +70,6 @@ jobs:
kill $(cat server.pid)
rm server.pid

- name: Remove latest installed platform version
Copy link
Member

Choose a reason for hiding this comment

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

I understand that these steps are not really necessary as they are taking care by the arduino-cli installation or?

@jaenrig-ifx
Copy link
Member

I mean, in general I would try to move asap all these things to the arduino-core-test and pull it in here.
The more we develop in parallel de more deviations and tech debt in future integration.

Then let´s try with the rest to get those workflows leaner, with some minimal calls to encapsulated functionality in scripts (that can also be run locally), etc.

But the first next step, I would say is to try to move all common content to the arduino-core-test, and consume that from this repo (and the xmc-for-arduino).

Anyhow, if the current hil workflow gets to work, we can just merge :)

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