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

Untangle node hierarchy #668

Merged
merged 8 commits into from
Aug 14, 2024

Conversation

raoulvdberge
Copy link
Contributor

No description provided.

@raoulvdberge raoulvdberge force-pushed the refactor/NO-ISSUE/untangle-node-hierarchy branch 3 times, most recently from 58cc9f6 to 4cf3962 Compare August 14, 2024 09:32
This can be moved to the base block entity instead.
…rkNodeContainerBlockEntity

This belongs in the base block entity instead.
Moved to base block entity instead. There is only one block that has
no redstone mode, and that one has a flag set.
This ensures that we can reuse it across the other block entities
and remove AbstractSchedulingNetworkNodeContainerBlockEntity.

* Move work tick rate to BaseNetworkNodeContainerBlockEntity.
* Move calculation of speed and work tick rate to UpgradeContainer.
@raoulvdberge raoulvdberge force-pushed the refactor/NO-ISSUE/untangle-node-hierarchy branch 2 times, most recently from 17fd7aa to 02f5f08 Compare August 14, 2024 10:14
@raoulvdberge raoulvdberge force-pushed the refactor/NO-ISSUE/untangle-node-hierarchy branch from 02f5f08 to dad98e0 Compare August 14, 2024 10:43
Copy link

sonarqubecloud bot commented Aug 14, 2024

Quality Gate Failed Quality Gate failed

Failed conditions
10.2% Duplication on New Code (required ≤ 10%)

See analysis details on SonarCloud

@raoulvdberge raoulvdberge merged commit 14442be into develop Aug 14, 2024
5 of 6 checks passed
@raoulvdberge raoulvdberge deleted the refactor/NO-ISSUE/untangle-node-hierarchy branch August 14, 2024 10:54
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 16, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant