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

Test suite: Creation of untagged images #16

Closed
wants to merge 2 commits into from

Conversation

ManiMatter
Copy link
Contributor

@ManiMatter ManiMatter commented May 27, 2024

  • loadImages function can now push images without setting a tag
  • Updated test cases accordingly

Disclaimer: Did not figure out(yet) how to test action locally (thus there might be bugs)
Proposed refinement: in the ci.yml, create a loop for single/multi to not repeat the test instructions (created a PR for it: #17)

- loadImages function can now push images without setting a tag
- Updated test cases accordingly

Disclaimer: Did not figure out(yet) how to test action locally (thus there might be bugs)
Proposed refinement: in the ci.yml, create a loop for single/multi to not repeat the test instructions
@rohanmars
Copy link
Contributor

Seems to be errors in the action.xml. See above. Also not sure why the ci.yml isn't running on the PR.

Were you able to run the ci.yml worflow on your local fork?

@rohanmars
Copy link
Contributor

Are you able to run the integration tests workflow in your fork?

@rohanmars
Copy link
Contributor

Oh, the ci.yml workflow is only setup to run on main. Change that the run on all branches. Then you'll be able to run ci workflow in your changes in your fork

@ManiMatter
Copy link
Contributor Author

No I was not able to test it - couldn't figure out how to approach it. Will look into the ci workflow, thanks for the pointer

@rohanmars
Copy link
Contributor

rohanmars commented May 27, 2024

Rebase your branch off the upstream main branch. I just main some changes which would enable the workflow on your repo.

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