Skip to content

Fix Deploy action#119

Merged
vodorok merged 2 commits intoEricsson:mainfrom
Discookie:ericsson/fix-deploy
Nov 4, 2022
Merged

Fix Deploy action#119
vodorok merged 2 commits intoEricsson:mainfrom
Discookie:ericsson/fix-deploy

Conversation

@Discookie
Copy link
Copy Markdown
Collaborator

Uses the Tests action as dependency instead of doing a copy of its own.

@Discookie Discookie added this to the 1.4.0 milestone Nov 2, 2022
@Discookie Discookie requested a review from vodorok November 2, 2022 01:25
Copy link
Copy Markdown
Collaborator

@vodorok vodorok left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please see my remark.

Comment thread .github/workflows/deploy.yml Outdated
uses: ./.github/workflows/test.yml
build:
name: Create extension
runs-on: ubuntu-18.04
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Github will deprecate the ubuntu-18.04 runner image pretty soon.
https://github.blog/changelog/2022-08-09-github-actions-the-ubuntu-18-04-actions-runner-image-is-being-deprecated-and-will-be-removed-by-12-1-22/
I thkink you could do the updates in one fell swoop.

Same for the test.yml

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Updated, and updated the various Actions we're using as well.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, looks good to me.

* Host system: Ubuntu 18 -> 22
* Node 12 -> 16
* Checkout v2 -> v3
@Discookie Discookie requested a review from vodorok November 3, 2022 11:34
Copy link
Copy Markdown
Collaborator

@vodorok vodorok left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@vodorok vodorok merged commit 0044458 into Ericsson:main Nov 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants