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.

Sign up for GitHub

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

Jump to bottom

.github/workflows: Added initial build support based on github actions #32

Merged
merged 2 commits into from
Oct 18, 2023
Merged

.github/workflows: Added initial build support based on github actions #32

merged 2 commits into from
Oct 18, 2023

Conversation

mphalke
Copy link
Collaborator

@mphalke mphalke commented Oct 17, 2023 •
edited
Loading

Created build .yaml file to build all the projects

@mphalke mphalke force-pushed the github-build-support branch 14 times, most recently from 20975e9 to 60e495e Compare October 17, 2023 08:43
@mphalke
.github/workflows: Added initial build support based on github actions
b57e117 
Created build .yaml file to build all the projects

Signed-off-by: MPhalke <[email protected]>
@mphalke mphalke force-pushed the github-build-support branch from 60e495e to b57e117 Compare October 17, 2023 08:58
jansunil
jansunil previously approved these changes Oct 17, 2023
View reviewed changes
@ribdp
Copy link
Contributor

ribdp commented Oct 17, 2023

If change is detected in any particular project folder, the build is triggered for that project

Is this implemented in the yaml file?

@mphalke
Copy link
Collaborator Author

mphalke commented Oct 17, 2023

If change is detected in any particular project folder, the build is triggered for that project

Is this implemented in the yaml file?

No, updated the comment now. Need to work on that yet

@CMinajigi
Copy link
Contributor

is this building projects now? How can we add matrix combination here.

@mphalke
Copy link
Collaborator Author

mphalke commented Oct 17, 2023

is this building projects now? How can we add matrix combination here.

is this building projects now? How can we add matrix combination here.

Yes all projects are building with default matrix combinations. Adding project based matrix combination is not in the scope of this PR and need to work on that in near future

@mphalke mphalke force-pushed the github-build-support branch from 9403fc3 to f6d942b Compare October 17, 2023 11:36
@mphalke
.github/worflows: Added daily build support
f1ed17e 
Added daily build github action workflow

Signed-off-by: MPhalke <[email protected]>
@mphalke mphalke force-pushed the github-build-support branch from f6d942b to f1ed17e Compare October 17, 2023 12:26
CMinajigi
CMinajigi approved these changes Oct 18, 2023
View reviewed changes
.github/workflows/build_projects.yaml
with:
name: ${{ matrix.project }}
path: |
projects/${{ matrix.project }}/build/${{ matrix.project }}.bin
Copy link
Contributor

Choose a reason for hiding this comment

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

where these builds are uploaded to.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

[github/workflows: Added initial build support based on github actions · analogdevicesinc/precision-converters-firmware@b57e117](https://github.com/analogdevicesinc/precision-converters-firmware/actions/runs/6547137835)

@mphalke
Copy link
Collaborator Author

mphalke commented Oct 18, 2023

Hi all, any other comments?

@mphalke mphalke merged commit 9878f9f into main Oct 18, 2023
19 checks passed
@mphalke mphalke deleted the github-build-support branch October 18, 2023 12:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@SaikiranGudla SaikiranGudla SaikiranGudla approved these changes

@CMinajigi CMinajigi CMinajigi approved these changes

@ribdp ribdp ribdp approved these changes

@jansunil jansunil jansunil approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@mphalke @ribdp @CMinajigi @jansunil @SaikiranGudla