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

CI Testing Pipeline #22

Merged
merged 102 commits into from
Dec 31, 2024
Merged

CI Testing Pipeline #22

merged 102 commits into from
Dec 31, 2024

Conversation

vincepick
Copy link
Collaborator

This Pull Request includes the addition of the CI pipeline and the testing scripts.

vincepick and others added 30 commits November 12, 2024 22:43
@vincepick
feat: added a new test script to scan output of AutoIG runs for key l…
fdd5717 
…ines
@vincepick
feat: added functional test script for macc-graded
07c4208
@vincepick
Create main.yml
2014aa6 
Starting to create a GitHub action to set up a CI workflow for automatically checking that previous functionalities are not broken by new PRs.
@vincepick
Merge pull request stacs-cp#15 from stacs-cp/vincepick-patch-1
a03a851 
Create main.yml for testing CI
@vincepick
fix: updated check.sh with exit scores for use by github actions
ec85b31
@vincepick
feat: updated main.yml to call check.sh
148c51e
@vincepick
Create testPush.yml
c3016cc
@vincepick
build: testing ability to make a PR which is picked up by github action
7080379
@vincepick
Merge pull request stacs-cp#16 from stacs-cp/testPRbranch
8de7fc3 
build: testing ability to make a PR which is picked up by github action
@vincepick
fix: updated branch to not run GitHub action on every PR, instead onl…
610841d 
…y those in test/maccGradedCase for now
@vincepick
build: trying to use docker volume within GitHub workflow
58f628a
@vincepick
fix: trying to directly use desired docker container in workflow
8da738f
@vincepick
build: temporary changing CI to always be wrong to check its actually…
66b5ffa 
… working
@vincepick
fix: undoing the intentional break
6d6bff9
@vincepick
build: breaking check script
36b629a
@vincepick
fix: undoing intentional break to check.sh
d4375f1
@vincepick
build: changing how docker container is used inside github actions
885f456
@vincepick
build: added updated dockerfile to this branch for use to build conta…
e7345d0 
…iner in GH workflow
@vincepick
build: updated yml file for building docker
004803d
@vincepick
build: updated use of root directory in workflow
080b5ec
@vincepick
fix: commenting out pushcontainer github workflow
f9e4043
@vincepick
fix: commenting out main.yml github workflow
3832c12
@vincepick
fix: updated testPush with more accurate job names
faa2517
@vincepick
fix: fixed testing push workflow again
2170136
@vincepick
build: added updated dockerfile
3b593fb
@vincepick
build: updating yaml for using container
23ca488
@vincepick
docs: updated tests readme
6ce72a0
@vincepick
build: updated docker push workflow
ccca4d1
@vincepick
Delete .github/workflows/main.yml
0baffb4
@vincepick
build: updated the workflow for automatically adding pushContainer to…
8fe0ba6 
… package again
@vincepick vincepick linked an issue Dec 19, 2024 that may be closed by this pull request
Test cases #25
Open
@vincepick vincepick marked this pull request as ready for review December 27, 2024 19:47
@vincepick vincepick requested a review from ndangtt December 27, 2024 20:25
@ndangtt
Copy link
Contributor

ndangtt commented Dec 30, 2024

@vincepick: thank you very much for creating the tests and the PR! To aid the review process and help future developers of this repo, could you please help add a summary of the tests created and the scripts? Many thanks!

@vincepick
Copy link
Collaborator Author

vincepick commented Dec 30, 2024 •
edited
Loading

@vincepick: thank you very much for creating the tests and the PR! To aid the review process and help future developers of this repo, could you please help add a summary of the tests created and the scripts? Many thanks!

Hi @ndangtt ,

Yes of course, I would be happy to. There is the existing summary in the testreadme.md file, and I would be happy to elaborate, but am not entirely sure what parts I should be focusing on. Could you let me know which specific areas you'd like me to touch on further? I will add them to the testreadme as well right away.

Thank you,
Vincent

@ndangtt ndangtt merged commit cf58c77 into stacs-cp:dev Dec 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ndangtt ndangtt Awaiting requested review from ndangtt

Assignees

@vincepick vincepick

Labels
enhancement New feature or request
Projects
None yet
Milestone
Macc Graded Automated Test Case
Development

Successfully merging this pull request may close these issues.

Test cases
3 participants
@vincepick @ozgurakgun @ndangtt