TappyPlane (QuickWings rewrite/fix) #5

Open
brxxh wants to merge 13 commits from brxxh/paradiso:tappyplane into main
First-time contributor

Renamed it to TappyPlane so it matches with the new assets from Kenney.

This is basically all I did:

  • No (extremely) magic values
    • There are some values that are explained by a comment above.
  • No global variables
    • There are still some constants that are global for "configuration"
  • No constructors
  • Clear code/logic
  • "Proper" collision detection using AABB
  • New assets
  • New structure (with a small "Asset Catalog")
Renamed it to TappyPlane so it matches with the new assets from Kenney. This is basically all I did: - No (extremely) magic values - There are some values that are explained by a comment above. - No global variables - There are still some constants that are global for "configuration" - No constructors - Clear code/logic - "Proper" collision detection using AABB - New assets - New structure (with a small "Asset Catalog")
brxxh added 13 commits 2025-07-02 12:30:53 +02:00
This pull request can be merged automatically.
You are not authorized to merge this pull request.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u tappyplane:brxxh-tappyplane
git checkout brxxh-tappyplane

Merge

Merge the changes and update on Forgejo.

Warning: The "Autodetect manual merge" setting is not enabled for this repository, you will have to mark this pull request as manually merged afterwards.

git checkout main
git merge --no-ff brxxh-tappyplane
git checkout brxxh-tappyplane
git rebase main
git checkout main
git merge --ff-only brxxh-tappyplane
git checkout brxxh-tappyplane
git rebase main
git checkout main
git merge --no-ff brxxh-tappyplane
git checkout main
git merge --squash brxxh-tappyplane
git checkout main
git merge --ff-only brxxh-tappyplane
git checkout main
git merge brxxh-tappyplane
git push origin main
Sign in to join this conversation.
No reviewers
No labels
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: Hartmut/paradiso#5
No description provided.