Skip to content

JmeSurfaceView: Package migration#1808

Closed
pavly-gerges wants to merge 14 commits into
jMonkeyEngine:masterfrom
Electrostat-Lab:package-migration
Closed

JmeSurfaceView: Package migration#1808
pavly-gerges wants to merge 14 commits into
jMonkeyEngine:masterfrom
Electrostat-Lab:package-migration

Conversation

@pavly-gerges
Copy link
Copy Markdown
Contributor

@pavly-gerges pavly-gerges commented May 4, 2022

This PR includes :

  • Migration to new com.jme3.view.surfaceview package.
  • Addition of package-info.java for the new packages.
  • Update of jme3-copyright.

This PR targets :

Note : this branch is a sub-branch of javadocs-format-improves, so it pre-resolves the conflicts after merging the docs improvements PR 👍 .

@pavly-gerges
Copy link
Copy Markdown
Contributor Author

Hmm, didn't expect these files to cause conflicts between the 2 branches....i guess nothing can resolve this, i need to copy my files to another new branch with new PR, because if i merged the 2 branches on top of one another, all the work would disappear.

@pavly-gerges
Copy link
Copy Markdown
Contributor Author

I tried to resolve the conflicts but i cannot, i will just close this and open a new clean one with the same changes (just copying files to another branch), i had to study later why this conflict happens even though i have refractored moved the classes to a new package at the top of this branch.

@pavly-gerges pavly-gerges deleted the package-migration branch May 13, 2022 07:33
@pavly-gerges
Copy link
Copy Markdown
Contributor Author

Ah i got it, i was sub-branching without updating the traffic automatically using git pull https://github.com/organization/repo.git parent-branch, my fault.

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.

2 participants