Skip to content

Fix the Travis JS related tests - #300

Merged
peternewman merged 24 commits into
OpenLightingProject:masterfrom
peternewman:fix-travis
Oct 4, 2026
Merged

peternewman merged 24 commits into
OpenLightingProject:masterfrom
peternewman:fix-travis

Conversation

@peternewman

Copy link
Copy Markdown
Member

No description provided.

@daveol daveol left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Overall seems fine, some minor nitpicks

Comment thread .travis.yml
Comment thread .travis.yml
Commented out the installation of grunt-cli for certain tasks.
Add MOZ_HEADLESS environment variable for headless testing.
Run Karma JS tests via GitHub Actions
Add lint and closure-compiler tasks to CI workflow
Removed implemented tasks from the commented list
Update task matrix in test workflow
@peternewman

Copy link
Copy Markdown
Member Author

I'm going to merge as-is, because basically everyone had approved this and the delta of changes was just the minor bits to get our new CI of GitHub Actions to run it:
7b08b9d...peternewman:rdm-app:fix-travis

This PR had been around so long it was still targeting Travis!

@peternewman
peternewman merged commit cff030a into OpenLightingProject:master Oct 4, 2026
4 of 5 checks passed
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.

4 participants