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

CI fails when using version 'a7683763fa44598d9faf6157a27632def618ad42' #223

Open
bcmhercun opened this issue Feb 13, 2024 · 13 comments
Open

Comments

@bcmhercun
Copy link

image

I'm aware that 'a7683763fa44598d9faf6157a27632def618ad42' version is not a release but I wanted to update my pipeline to get rid of Node.js 16 deprecation warning.

Maybe someone (@jurplel ) is aware of this issue already but I'm leaving it here just in case.

@pzhlkj6612
Copy link
Contributor

Hi.

You can build this action yourself. This is how CI does it:

steps:
- uses: actions/checkout@v4
- uses: actions/setup-node@v4
with:
node-version: 20
cache: npm
cache-dependency-path: action/
- run: |
cd action
npm ci || npm install
shell: bash
- run: |
cd action
npm run build
- name: Install Qt with options and default aqtversion
if: ${{ !matrix.aqtversion && matrix.qt.version }}
uses: ./
with:
modules: ${{ matrix.qt.modules }}
version: ${{ matrix.qt.requested }}
tools: tools_ifw tools_qtcreator,qt.tools.qtcreator
cache: ${{ matrix.cache == 'cached' }}

For your purpose:

    steps:
      - uses: actions/checkout@v4
        with:
          repository: 'jurplel/install-qt-action'
          ref: 'a7683763fa44598d9faf6157a27632def618ad42'
          path: 'install-qt-action'

      - uses: actions/setup-node@v4
        with:
          node-version: 20
          cache: npm
          cache-dependency-path: install-qt-action/action/

      - name: Build jurplel/install-qt-action
        run: |
          cd install-qt-action/action/
          npm ci || npm install
          npm run build
        shell: bash

      - name: Install Qt
        uses: install-qt-action/
        with:
          modules: ...
          version: ...
          tools: ...
          cache: ...

Append your own steps then everything should be OK.

@bcmhercun
Copy link
Author

bcmhercun commented Feb 14, 2024

@pzhlkj6612
image
It is not pointing to the line where error is occurring.
I'm assuming it is about
uses: install-qt-action/

EDITED:
uses: ./install-qt-action/action/

fixed the issue (kinda obvious but I missed it at first glance :D

@pzhlkj6612
Copy link
Contributor

I'm assuming it is about uses: install-qt-action/

EDITED: uses: ./install-qt-action/action/

fixed the issue

My bad, you're right.


Does your own built action function normally?

@bcmhercun
Copy link
Author

I'm assuming it is about uses: install-qt-action/
EDITED: uses: ./install-qt-action/action/
fixed the issue

My bad, you're right.

Does your own built action function normally?

@pzhlkj6612 Yes, after this change in uses: it works well ;)

@bcmhercun
Copy link
Author

@pzhlkj6612 unfortunately another deprecation warning has occured:

image

In this step:

    - name: Install Qt
      uses: ./install-qt-action/action/
      with:
        version: 6.2.4
        dir: ${{ env.QT_INSTALL_DIR }}
        cache: true

There's no direct call to set-output. Does anybody have idea how to trace it?

@Wissididom
Copy link

Wissididom commented Feb 15, 2024

@pzhlkj6612 unfortunately another deprecation warning has occured:

image

In this step:

    - name: Install Qt
      uses: ./install-qt-action/action/
      with:
        version: 6.2.4
        dir: ${{ env.QT_INSTALL_DIR }}
        cache: true

There's no direct call to set-output. Does anybody have idea how to trace it?

For that I think this repos just needs to update @actions/core to at least 1.10.0 (See https://github.com/jurplel/install-qt-action/blob/master/action/package.json and https://github.com/actions/toolkit/blob/main/packages/core/RELEASES.md) - Preferable 1.10.1 since that currently is latest

@pzhlkj6612
Copy link
Contributor

For that I think this repos just needs to update @actions/core to at least 1.10.0 (See https://github.com/jurplel/install-qt-action/blob/master/action/package.json and https://github.com/actions/toolkit/blob/main/packages/core/RELEASES.md) - Preferable 1.10.1 since that currently is latest

@Wissididom , I think you are right.

"@actions/core": "^1.9.1",

1.10.0

  • saveState and setOutput now use environment files if available #1178
  • getMultilineInput now correctly trims whitespace by default #1185

https://github.com/actions/toolkit/blob/415c42d27ca2a24f3801dd9406344aaea00b7866/packages/core/RELEASES.md#1100

@jurplel maybe we need to update the dependency.

@jurplel
Copy link
Owner

jurplel commented Feb 16, 2024

I'll make a release

@jurplel
Copy link
Owner

jurplel commented Feb 16, 2024

I am having some test failures on the v3 branch. I need to find time to debug it before I can make a release

@bcmhercun
Copy link
Author

bcmhercun commented Mar 4, 2024

@jurplel this also needs to be updated to 4.0.0

"@actions/cache": "^3.0.0",

@Wissididom
Copy link

this also needs to be updated to 4.0.0

"@actions/cache": "^3.0.0",

I don't think so. According to https://github.com/actions/toolkit/blob/main/packages/cache/RELEASES.md and https://www.npmjs.com/package/@actions/cache 3.2.4 is the latest version.

I think it needs to be updated to at least 3.0.5 tho since that's what they write in their CHANGELOG.md for that version:

Update `@actions/cache` to use `@actions/core@^1.10.0`

@bcmhercun
Copy link
Author

this also needs to be updated to 4.0.0

"@actions/cache": "^3.0.0",

I don't think so. According to https://github.com/actions/toolkit/blob/main/packages/cache/RELEASES.md and https://www.npmjs.com/package/@actions/cache 3.2.4 is the latest version.

I think it needs to be updated to at least 3.0.5 tho since that's what they write in their CHANGELOG.md for that version:

Update `@actions/cache` to use `@actions/core@^1.10.0`

image
image

@Wissididom
Copy link

Wissididom commented Mar 6, 2024

this also needs to be updated to 4.0.0

"@actions/cache": "^3.0.0",

I don't think so. According to actions/toolkit@main/packages/cache/RELEASES.md and npmjs.com/package/@actions/cache 3.2.4 is the latest version.
I think it needs to be updated to at least 3.0.5 tho since that's what they write in their CHANGELOG.md for that version:

Update `@actions/cache` to use `@actions/core@^1.10.0`

image image

Then I think I got the Cache Action that is being used as a standalone action and the cache action, that is part of the toolkit to develop actions with, mixed up. On npm was the actions/cache of toolkit (where I started looking) and you linked the actual actions/cache action hosted on GitHub.

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

4 participants