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

Static Code Analyzer for Unused Code #74

Open
1 task done
PSchmiedmayer opened this issue Jun 7, 2024 · 1 comment
Open
1 task done

Static Code Analyzer for Unused Code #74

PSchmiedmayer opened this issue Jun 7, 2024 · 1 comment
Labels
enhancement New feature or request

Comments

@PSchmiedmayer
Copy link
Member

Problem

Some of the Spezi-based applications contain a decent amount of dead code as students or inexperienced developers forget to remove functions, unused views, and leftover dependencies/import statements.

Solution

We should use a tool that provides a static code analyzer for unused code and ideally integrate this in a PR workflow. For languages like Swift, periphery might be a great solution.

Additional context

It might be great to see/investigate if periphery starts to support SPM/Xcode build plugins as a great way to integrate this right in the build workflow locally as well: peripheryapp/periphery#600

Code of Conduct

  • I agree to follow this project's Code of Conduct and Contributing Guidelines
@PSchmiedmayer PSchmiedmayer added the enhancement New feature or request label Jun 7, 2024
@PSchmiedmayer PSchmiedmayer moved this to Focus Areas in Project Planning Jun 7, 2024
@PSchmiedmayer
Copy link
Member Author

Periphery also introduced a nice formatter for GitHub Actions: peripheryapp/periphery#746; a very nice integration!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant