Skip to content

Remove non-working Banana Pi code#1252

Merged
uweseimet merged 5 commits intodevelopfrom
issue_1248
Oct 22, 2023
Merged

Remove non-working Banana Pi code#1252
uweseimet merged 5 commits intodevelopfrom
issue_1248

Conversation

@uweseimet
Copy link
Copy Markdown
Contributor

Successfully tested with a Pi 4.

@uweseimet uweseimet linked an issue Oct 17, 2023 that may be closed by this pull request
@uweseimet uweseimet requested a review from dialtr October 17, 2023 08:50
@uweseimet uweseimet marked this pull request as ready for review October 17, 2023 09:26
@sonarqubecloud
Copy link
Copy Markdown

SonarCloud Quality Gate failed.    Quality Gate failed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 1 Code Smell

0.0% 0.0% Coverage
1.1% 1.1% Duplication

idea Catch issues before they fail your Quality Gate with our IDE extension sonarlint SonarLint

@uweseimet
Copy link
Copy Markdown
Contributor Author

This PR is blocking further PRs, e.g. for tickets like #1092, because these tickets require this PR to be merged first in order to save me from a code conflict hell.
@rdmark @dialtr Any help is appreciated. Since this ticket essentially only removes unused code, just testing whether piscsi still works on a Raspberry Pi should be sufficient. I have already tested this for the Pi 4.

Copy link
Copy Markdown
Collaborator

@dialtr dialtr left a comment

Choose a reason for hiding this comment

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

LGTM. I also grepped the source for any references to allwinner/banana that may have been missed and found none.

@uweseimet uweseimet merged commit 43088ab into develop Oct 22, 2023
@uweseimet uweseimet deleted the issue_1248 branch October 22, 2023 15:29
@uweseimet
Copy link
Copy Markdown
Contributor Author

Thank you @dialtr. This has paved the way for PR #1261.

uweseimet added a commit that referenced this pull request Oct 22, 2023
* Remove non-working Banana Pi code

* Remove SonarQube suppressions which can be resolved by a code update

* Update device detection
uweseimet added a commit that referenced this pull request Oct 22, 2023
* Remove non-working Banana Pi code

* Remove SonarQube suppressions which can be resolved by a code update

* Update device detection
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.

Discard non-working Banana Pi code

2 participants