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

Bump Phaser from v3.55.2 to v3.87.0 #3090

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

JustATin555
Copy link

@JustATin555 JustATin555 commented Feb 4, 2025

Description

  • Bump Phaser from 3.55.2 to 3.87.0 and address any breaking changes
  • Add path as explicit dev dependency as Phaser 3.87.0 no longer lists it
  • Add phaser3spectorjs to satisfy Phaser 3.87.0 undeclared dependency (unknown problem with Phaser's build process)
  • Add a guard for scene.input.mouse or scene.input.keyboard not existing

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update
  • Code quality improvements

How to test

  • Tested with yarn test, all test cases passed
  • Tested with yarn start, Source Academy launches and mock story chapters load properly
  • Tested with actual story chapters

Checklist

  • I have tested this code
  • I have updated the documentation

@JustATin555 JustATin555 self-assigned this Feb 4, 2025
@JustATin555 JustATin555 linked an issue Feb 4, 2025 that may be closed by this pull request
@coveralls
Copy link

coveralls commented Feb 4, 2025

Pull Request Test Coverage Report for Build 13140973888

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 31.196%

Totals Coverage Status
Change from base Build 13133948412: 0.0%
Covered Lines: 4842
Relevant Lines: 14646

💛 - Coveralls

@JustATin555 JustATin555 changed the title Bump phaser justatin555 Bump Phaser 3.55.2 to 3.87.0 Feb 4, 2025
@JustATin555 JustATin555 added dependencies This pull request updates dependency files game All game related ideas, bugs, fixes labels Feb 4, 2025
@JustATin555 JustATin555 changed the title Bump Phaser 3.55.2 to 3.87.0 Bump Phaser from v3.55.2 to v3.87.0 Feb 4, 2025
@RichDom2185
Copy link
Member

Supersedes #3087

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies This pull request updates dependency files game All game related ideas, bugs, fixes
Development

Successfully merging this pull request may close these issues.

Bump Phaser from 3.55.2 to 3.87.0
3 participants