Skip to content
Snippets Groups Projects

ViewHelper code tidy

Merged Benjamin Michael Wynne requested to merge bwynne/athena:ViewHelperTidy into master

Fixing whitespace and formatting, since this is supposed to be a "helper" class and therefore it's important to keep it readable.

Also removing a bunch of unused code and methods, on the assumption that by now we've stabilised on correct usage patterns and should guide people towards them rather than providing alternatives.

Edited by Benjamin Michael Wynne

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • This merge request affects 1 package:

    • Control/AthViews

    Adding @ssnyder as watcher

  • Benjamin Michael Wynne resolved all threads

    resolved all threads

  • added 2 commits

    • ab168e5a - Additional tidying. Renaming of methods/arguments with lower case first letter
    • dcf7be32 - Update the EVCAs

    Compare with previous version

  • Benjamin Michael Wynne resolved all threads

    resolved all threads

  • This merge request affects 2 packages:

    • Control/AthViews
    • Trigger/TrigSteer/ViewAlgs

    Adding @tamartin ,@ssnyder as watchers

  • It all looks good to me. Thanks Ben

  • :white_check_mark: CI Result SUCCESS (hash 34a76146)

    Athena AthSimulation
    externals :white_check_mark: :white_check_mark:
    cmake :white_check_mark: :white_check_mark:
    make :white_check_mark: :white_check_mark:
    required tests :white_check_mark: :white_check_mark:
    optional tests :white_check_mark: :white_check_mark:

    Full details available on this CI monitor view
    :white_check_mark: Athena: number of compilation errors 0, warnings 0
    :white_check_mark: AthSimulation: number of compilation errors 0, warnings 0
    :pencil: For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 10796]

  • :white_check_mark: CI Result SUCCESS (hash dcf7be32)

    Athena AthSimulation
    externals :white_check_mark: :white_check_mark:
    cmake :white_check_mark: :white_check_mark:
    make :white_check_mark: :white_check_mark:
    required tests :white_check_mark: :white_check_mark:
    optional tests :white_check_mark: :white_check_mark:

    Full details available on this CI monitor view
    :white_check_mark: Athena: number of compilation errors 0, warnings 0
    :white_check_mark: AthSimulation: number of compilation errors 0, warnings 0
    :pencil: For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 10807]

  • added 101 commits

    Compare with previous version

  • This merge request affects 2 packages:

    • Control/AthViews
    • Trigger/TrigSteer/ViewAlgs

    Adding @tamartin ,@ssnyder as watchers

  • added review-pending-level-1 label and removed review-approved label

  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading