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

Allow using fly instead of only flyctl #44

Open
einarpersson opened this issue Apr 27, 2023 · 4 comments
Open

Allow using fly instead of only flyctl #44

einarpersson opened this issue Apr 27, 2023 · 4 comments

Comments

@einarpersson
Copy link

Hi!

I just had a problem where I got fly: command not found. When I changed to flyctl it started working.

In your official documentation you use fly instead of flyctl quite a lot. I would therefore recommend that using fly would also be OK in workflows using this action. I think it should be consistent with the normal installation (which adds the symlink fly->flyctl).

@CanRau
Copy link
Contributor

CanRau commented May 21, 2023

same just happened to, was confusing as I thought the CLI is moving more towards fly instead of flyctl so stopped using the latter completely 🥲

@Igloczek
Copy link

For those who don't want to wait, or just prefer full CLI access, drop it in your workflow config instead of official action, and you are good to go.

      - name: Install Fly CLI
        run: |
          curl -L https://fly.io/install.sh | sh
          echo "/home/runner/.fly/bin" >> $GITHUB_PATH

If you are not using runs-on: ubuntu-latest you most likely will need to update the directory that is added to PATH, based on what the setup script will print in logs.

@roeniss
Copy link

roeniss commented Aug 15, 2023

So here the issue...

Is there any specific reason for removing fly? 👀

@rockse
Copy link

rockse commented Oct 1, 2023

Any update on when this will be updated?

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

No branches or pull requests

5 participants