Skip to content

Conversation

@lspepinho
Copy link

This PR adds simple backwards compatibility support, exactly as described from Fixing Lua Mods from 0.6.x to 1.0 in the original (now archived) PsychEngine repository

@lspepinho lspepinho changed the title Add compatibility layer for 0.6.3 mods Add compatibility layer for 0.6.3 mods in 1.0.4 Sep 12, 2025
@mikolka9144
Copy link
Collaborator

This might be peak

@lspepinho
Copy link
Author

This is peak.

@mikolka9144 mikolka9144 changed the base branch from master to master-dev October 4, 2025 16:17
@mikolka9144
Copy link
Collaborator

You need to update your branch with the dev version

@lspepinho
Copy link
Author

oh, imma do it

Copy link
Collaborator

@mikolka9144 mikolka9144 left a comment

Choose a reason for hiding this comment

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

Looks mostly good, just please revert unnecessary changes intorduced in your PR

Copy link
Collaborator

Choose a reason for hiding this comment

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

Same as above

Copy link
Author

Choose a reason for hiding this comment

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

oh, i accidentally merged the latest p-slice

Copy link
Collaborator

Choose a reason for hiding this comment

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

Not sure what happened here, but could you revert this change?

{
public function new(x:Float, y:Float)
{
super(x, y, "charSelect/bfChill");
Copy link
Collaborator

Choose a reason for hiding this comment

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

I belive this could break some things outside of the Psych 0.6.3 compatiblity

Copy link
Collaborator

@mikolka9144 mikolka9144 Oct 8, 2025

Choose a reason for hiding this comment

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

If I remember correctly this class was moved, so no point in adding it back

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.

6 participants