Skip to content

CHANGE: Update CinemachineV2 to CinemachineV3#2384

Merged
josepmariapujol-unity merged 5 commits intodevelopfrom
input/upgrade-cinemachine-v3
Mar 17, 2026
Merged

CHANGE: Update CinemachineV2 to CinemachineV3#2384
josepmariapujol-unity merged 5 commits intodevelopfrom
input/upgrade-cinemachine-v3

Conversation

@josepmariapujol-unity
Copy link
Collaborator

@josepmariapujol-unity josepmariapujol-unity commented Mar 17, 2026

Description

Update from CinemachineV2 to CinemachineV3 to fix internal CI errors:

Screenshot 2026-03-17 at 10 55 38

Before:
Screenshot 2026-03-17 at 11 31 37

After:
image

Testing status & QA

Sample test video manually tested:
https://github.com/user-attachments/assets/3d08a71a-f7f2-4c47-9e3b-8159b171783c

Overall Product Risks

Please rate the potential complexity and halo effect from low to high for the reviewers. Note down potential risks to specific Editor branches if any.

  • Complexity: 1
  • Halo Effect: 1

Comments to reviewers

Please describe any additional information such as what to focus on, or historical info for the reviewers.

Checklist

Before review:

  • Changelog entry added.
    • Explains the change in Changed, Fixed, Added sections.
    • For API change contains an example snippet and/or migration example.
    • JIRA ticket linked, example (case %%). If it is a private issue, just add the case ID without a link.
    • Jira port for the next release set as "Resolved".
  • Tests added/changed, if applicable.
    • Functional tests Area_CanDoX, Area_CanDoX_EvenIfYIsTheCase, Area_WhenIDoX_AndYHappens_ThisIsTheResult.
    • Performance tests.
    • Integration tests.
  • Docs for new/changed API's.
    • Xmldoc cross references are set correctly.
    • Added explanation how the API works.
    • Usage code examples added.
    • The manual is updated, if needed.

During merge:

  • Commit message for squash-merge is prefixed with one of the list:
    • NEW: ___.
    • FIX: ___.
    • DOCS: ___.
    • CHANGE: ___.
    • RELEASE: 1.1.0-preview.3.

@josepmariapujol-unity josepmariapujol-unity self-assigned this Mar 17, 2026
@josepmariapujol-unity josepmariapujol-unity marked this pull request as ready for review March 17, 2026 10:11
@josepmariapujol-unity josepmariapujol-unity changed the title Input/upgrade cinemachine v3 CHANGE: Update CinemachineV2 to CinemachineV3 Mar 17, 2026
Copy link
Contributor

@u-pr u-pr bot left a comment

Choose a reason for hiding this comment

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

May require changes
This PR introduces a significant version mismatch between the Cinemachine API used in the code and the package version specified in the manifest, which will lead to compilation errors. There are also potential issues with broken serialized field references due to the type change.

🤖 Helpful? 👍/👎

@codecov-github-com
Copy link

codecov-github-com bot commented Mar 17, 2026

Codecov Report

All modified and coverable lines are covered by tests ✅

@@            Coverage Diff            @@
##           develop    #2384    +/-   ##
=========================================
  Coverage    77.90%   77.91%            
=========================================
  Files          476      482     +6     
  Lines        97613    97756   +143     
=========================================
+ Hits         76048    76162   +114     
- Misses       21565    21594    +29     
Flag Coverage Δ
inputsystem_MacOS_2022.3 5.49% <ø> (-0.04%) ⬇️
inputsystem_MacOS_2022.3_project 75.42% <ø> (+0.02%) ⬆️
inputsystem_MacOS_6000.0 5.27% <ø> (-0.04%) ⬇️
inputsystem_MacOS_6000.0_project 77.33% <ø> (+0.03%) ⬆️
inputsystem_MacOS_6000.3 5.27% <ø> (-0.04%) ⬇️
inputsystem_MacOS_6000.3_project 77.32% <ø> (+0.03%) ⬆️
inputsystem_MacOS_6000.4 5.28% <ø> (-0.04%) ⬇️
inputsystem_MacOS_6000.6_project 77.33% <ø> (+0.02%) ⬆️
inputsystem_Ubuntu_2022.3_project 75.22% <ø> (+0.02%) ⬆️
inputsystem_Ubuntu_6000.0 5.28% <ø> (-0.04%) ⬇️
inputsystem_Ubuntu_6000.0_project 77.13% <ø> (+0.03%) ⬆️
inputsystem_Ubuntu_6000.3 5.28% <ø> (-0.04%) ⬇️
inputsystem_Ubuntu_6000.3_project 77.13% <ø> (+0.02%) ⬆️
inputsystem_Ubuntu_6000.4 5.28% <ø> (-0.04%) ⬇️
inputsystem_Ubuntu_6000.4_project 77.14% <ø> (+0.02%) ⬆️
inputsystem_Ubuntu_6000.6 5.28% <ø> (-0.04%) ⬇️
inputsystem_Ubuntu_6000.6_project 77.13% <ø> (+0.02%) ⬆️
inputsystem_Windows_2022.3 5.50% <ø> (-0.04%) ⬇️
inputsystem_Windows_2022.3_project 75.55% <ø> (+0.02%) ⬆️
inputsystem_Windows_6000.0 5.27% <ø> (-0.04%) ⬇️
inputsystem_Windows_6000.0_project 77.45% <ø> (+0.02%) ⬆️
inputsystem_Windows_6000.3 5.27% <ø> (-0.04%) ⬇️
inputsystem_Windows_6000.3_project 77.45% <ø> (+0.02%) ⬆️
inputsystem_Windows_6000.4 5.28% <ø> (-0.04%) ⬇️
inputsystem_Windows_6000.4_project 77.45% <ø> (+0.02%) ⬆️
inputsystem_Windows_6000.6 5.28% <ø> (-0.04%) ⬇️
inputsystem_Windows_6000.6_project 77.45% <ø> (+0.02%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

see 7 files with indirect coverage changes

@Pauliusd01
Copy link
Collaborator

Won't this break 22.3 and possibly 6.0? Don't have the time to check yet but my first thought

@josepmariapujol-unity
Copy link
Collaborator Author

Won't this break 22.3 and possibly 6.0? Don't have the time to check yet but my first thought

So far, only 6.5 test are failing due to coding obsolete API. Worth if you could check it out.

@josepmariapujol-unity
Copy link
Collaborator Author

No errors:
Screenshot 2026-03-17 at 13 21 16

@josepmariapujol-unity
Copy link
Collaborator Author

No errors (22.3):
Screenshot 2026-03-17 at 13 25 46

@josepmariapujol-unity josepmariapujol-unity merged commit b06cc33 into develop Mar 17, 2026
110 of 132 checks passed
@josepmariapujol-unity josepmariapujol-unity deleted the input/upgrade-cinemachine-v3 branch March 17, 2026 12:31
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.

3 participants