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 basic GHA jobs #2688

Open
wants to merge 12 commits into
base: main
Choose a base branch
from
Open

Add basic GHA jobs #2688

wants to merge 12 commits into from

Conversation

johnwparent
Copy link
Collaborator

@johnwparent johnwparent commented Jul 17, 2024

Converts existing GHA job to new workflow model

Adds build jobs for Windows, Linux, and MacOS

Describe your changes

Issue ticket number and link

Checklist before requesting a review

  • I have performed a self-review of my code.
  • If it is a core feature, I have added thorough tests.
  • I have added thorough documentation for my code.
  • I have tagged PR with relevant project labels. I acknowledge that a PR without labels may be dismissed.
  • If this PR addresses a bug, I have provided both a screenshot/video of the original bug and the working solution.

Demo

Steps to Reproduce

Notes

Fixes #2593

Converts existing GHA job to new workflow model

Adds build jobs for Windows, Linux, and MacOS

Signed-off-by: John Parent <[email protected]>
@manyoso
Copy link
Collaborator

manyoso commented Jul 17, 2024

Where is ./.github/workflows/build-chat.yml

Signed-off-by: John Parent <[email protected]>
@johnwparent
Copy link
Collaborator Author

I did a git add *, apparently build-chat.yml is getting hit by a .gitignore... none of my git commands consider it.
Force added and pushed.

Signed-off-by: John Parent <[email protected]>
@johnwparent johnwparent force-pushed the gha/add-basic-automatic-ci branch from 9c66fd8 to cc643d0 Compare July 18, 2024 22:55
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.

Implement at least minimal *automatic* CI
2 participants