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

testing shortening PR workflow #69

Merged
merged 2 commits into from Feb 18, 2022
Merged

testing shortening PR workflow #69

merged 2 commits into from Feb 18, 2022

Conversation

vsoch
Copy link
Contributor

@vsoch vsoch commented Feb 17, 2022

Signed-off-by: vsoch vsoch@users.noreply.github.com

@trws
Copy link
Collaborator

trws commented Feb 17, 2022

Are these build failures expected? It kinda looks like missing directories?

@vsoch
Copy link
Contributor Author

vsoch commented Feb 17, 2022

I think what is happening is that one is generated in the container and the other outside, so they are different - checking now with an echo!

@vsoch
Copy link
Contributor Author

vsoch commented Feb 17, 2022

yep, behold, path inside and outside the container! Another artifact of trying to do something outside the container first instead of letting uptodate run the show!

/home/runner/work/docker-images/docker-images
/home/runner/work/_temp/4b944797-321d-45d5-a48e-cebcd8ea1eff.sh: line 5: cd: /github/workspace/ubuntu/gcc: No such file or directory

@vsoch
Copy link
Contributor Author

vsoch commented Feb 17, 2022

Huh the build matrices shouldn't all be running like that without changed files - let me look into that too.

@vsoch
Copy link
Contributor Author

vsoch commented Feb 17, 2022

okay the issue was passing an empty string for parsed files (none found) which defaults to the PWD and will trigger all! I added another hack to fix it lol.

Signed-off-by: vsoch <vsoch@users.noreply.github.com>
Signed-off-by: vsoch <vsoch@users.noreply.github.com>
@trws trws merged commit 3cbf4c6 into main Feb 18, 2022
@vsoch vsoch deleted the shorten-workflows branch February 18, 2022 00:42
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.

None yet

3 participants