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

Remove confusing message about symlinks #381

Merged
merged 1 commit into from
Oct 6, 2020

Conversation

lyleunderwood
Copy link
Contributor

Fixes #209

It turns out that act does copy symlinks. I've setup a test case here:

lyleunderwood/github-actions-demo@d904f63

Below is the output. Notice the message, "Unable to copy link test --> README.md" followed shortly by the description and usage of the corresponding symlink that exists within the workspace.

lyle@lyle-t410:~/dev/github-actions-demo$ act
[CI/test] 🚀  Start image=node:12.6-buster-slim
[CI/test]   🐳  docker run image=node:12.6-buster-slim entrypoint=["/usr/bin/tail" "-f" "/dev/null"] cmd=[]
[CI/test]   🐳  docker cp src=/home/lyle/dev/github-actions-demo/. dst=/github/workspace
[CI/test] Unable to copy link test --> README.md
[CI/test] ⭐  Run actions/checkout@v2
[CI/test]   ✅  Success - actions/checkout@v2
[CI/test] ⭐  Run ls -l ./test
| lrwxrwxrwx 1 node node 9 Oct  2 23:51 ./test -> README.md
[CI/test]   ✅  Success - ls -l ./test
[CI/test] ⭐  Run head ./test
| # Overview
| Simple Node.js application to demonstrate the use of GitHub Actions
| 
| # Look Ma, no Makefile!
| All the tasks necessary for testing, building and deploying this code is already defined in `.github/workflows/` so why would you want to also create a `Makefile` for local development?  Now you can use [act](https://github.com/nektos/act) to run the actions locally!
| 
| Try these:
| 
| * `act -j lint` - run the linter
| * `act -j test` - run the tests
[CI/test]   ✅  Success - head ./test

@cplee cplee merged commit 3d9b825 into nektos:master Oct 6, 2020
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.

Doesn't copy symlinks
2 participants