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

Update GitHub actions, fix NPMv16 deprecation #488

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

thePanz
Copy link

@thePanz thePanz commented Oct 30, 2024

Update to use latest v4.x for:

  • actions/checkout → solves NPMv16 deprecation
  • actions/cache

Use latest Ubuntu, upgrade from ubuntu-20.04

Update to use latest v4 for:
- actions/checkout → solves NPMv16 deprecation
- actions/cache

Use latest Ubuntu, upgrade from ubuntu-20.04
@tobias-93
Copy link
Collaborator

If you can get the JS test running again I will merge this.

@thePanz
Copy link
Author

thePanz commented Nov 26, 2024

Hi @tobias-93 I tried but my got stuck 😞

I would need some help here, not sure why that test if failing, looks like the tool is not available or installed 🤷

@tobias-93
Copy link
Collaborator

Looks like both PhantomJS and the Google Closure Library which are being used for the JS unit tests are legacy tools which are no longer supported. This means that the JS tests need to be rewritten to a new library, for which Google (the developers, not the search engine) suggests Jasmine, Chai, Mocha, Jest, etc. This has no high priority for me, but feel free to see if you can rework the tests if you are familiar with any of these frameworks or if you feel like learning one of these.

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.

2 participants