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

chore: Use FlexiblePathfinding / Remove Pathfinding #88

Merged
merged 2 commits into from
Jan 13, 2022

Conversation

skaldarnar
Copy link
Contributor

  • remove minion.work package

  • remove grid package

  • use org.terasology.module.behaviors as root namespace

  • separate three packages actions, components, and systems

    • integrate nodes into actions as they are all behavior actions
    • rename system to systems to use consistent plural nouns
    • sort stray classes into respective package
  • remove irrelevant or broken tests (i.e., all tests)

      src/main/java/
      └── org
      		└── terasology
      				└── module
      						└── behaviors
      								├── actions
      								├── components
      								└── systems
    

- remove `minion.work` package
- remove `grid` package
- use `org.terasology.module.behaviors` as root namespace
- separate three packages `actions`, `components`, and `systems`
  - integrate `nodes` into `actions` as they are all behavior actions
  - rename `system` to `systems` to use consistent plural nouns
  - sort stray classes into respective package
- remove irrelevant or broken tests (i.e., all tests)

		src/main/java/
		└── org
				└── terasology
						└── module
								└── behaviors
										├── actions
										├── components
										└── systems
@jdrueckert jdrueckert marked this pull request as ready for review January 13, 2022 21:34
@jdrueckert jdrueckert changed the title [WIP] Use FlexiblePathfinding / Remove Pathfinding chore: Use FlexiblePathfinding / Remove Pathfinding Jan 13, 2022
skaldarnar added a commit to Terasology/GooeysQuests that referenced this pull request Jan 13, 2022
Adjust for refactorings done in Terasology/Behaviors#88.

The changes resulted from automatic refactorings in IntelliJ and are **untested**.
The IDE support only helps with Java classes, so there might be assets or other references that slipped through.

At least Omega compiles with these changes.
skaldarnar added a commit to Terasology-Archived/GooKeeper that referenced this pull request Jan 13, 2022
Adjust for refactorings done in Terasology/Behaviors#88.

The changes resulted from automatic refactorings in IntelliJ and are **untested**.
The IDE support only helps with Java classes, so there might be assets or other references that slipped through.

At least Omega compiles with these changes.
skaldarnar added a commit to Terasology/MasterOfOreon that referenced this pull request Jan 13, 2022
Adjust for refactorings done in Terasology/Behaviors#88.

The changes resulted from automatic refactorings in IntelliJ and are **untested**.
The IDE support only helps with Java classes, so there might be assets or other references that slipped through.

At least Omega compiles with these changes.
skaldarnar added a commit to Terasology/MetalRenegades that referenced this pull request Jan 13, 2022
Adjust for refactorings done in Terasology/Behaviors#88.

The changes resulted from automatic refactorings in IntelliJ and are **untested**.
The IDE support only helps with Java classes, so there might be assets or other references that slipped through.

At least Omega compiles with these changes.
skaldarnar added a commit to Terasology/WildAnimals that referenced this pull request Jan 13, 2022
Adjust for refactorings done in Terasology/Behaviors#88.

The changes resulted from automatic refactorings in IntelliJ and are **untested**.
The IDE support only helps with Java classes, so there might be assets or other references that slipped through.

At least Omega compiles with these changes.
skaldarnar added a commit to Terasology/WildAnimalsGenome that referenced this pull request Jan 13, 2022
Adjust for refactorings done in Terasology/Behaviors#88.

The changes resulted from automatic refactorings in IntelliJ and are **untested**.
The IDE support only helps with Java classes, so there might be assets or other references that slipped through.

At least Omega compiles with these changes.
@jdrueckert jdrueckert merged commit d3600d1 into develop Jan 13, 2022
@jdrueckert jdrueckert deleted the feat/merge-flexible-movement branch January 13, 2022 22:13
@jdrueckert
Copy link
Member

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