Sitelet https://github.com/errbit/errbit/issues/3125
Skip to content

Review potentially unused Errbit code identified by static analysis #3125

Description

@mikelolasagasti

unused 0.4.0 reported the following 19 high-likelihood candidates across 9 files. These are suggestions requiring manual verification, not confirmed dead code.

Findings

  • spec/models/error_report_spec.rb
    • API_VERSION
  • bin/rails
    • APP_PATH
  • app/models/problem.rb
    • attribute_count_decrease
    • merged?
  • app/models/user.rb
    • can_create_github_issues?
    • github_login=
    • password_required?
    • watching?
  • app/models/app.rb
    • find_by_id!
  • spec/errbit_plugin/mock_issue_tracker.rb
    • comments_allowed?
  • app/assets/stylesheets/mailers/mailer.css
    • a.bold
    • p.heading
    • p.monospace
    • span.bold
  • app/assets/stylesheets/reset.css
    • applet
    • dfn
    • strike
  • app/assets/stylesheets/bootstrap.css
    • form-control-lg::-webkit-file-upload-button
    • form-control-sm::-webkit-file-upload-button

Suggested follow-up

Review each candidate against Rails conventions, callbacks, reflection, view usage, routes, JavaScript usage, and test support before removing anything.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions