Skip to content
Snippets Groups Projects

Simplify Algorithm constructor

Merged Gerhard Raven requested to merge graven/Gaudi:simplify-algorithm-constructor into master
All threads resolved!
  • move determination of auditor default to dedicated function, and use that in the declaration of the auditor properties
  • see !765 (comment 1855312)

Merge request reports

Pipeline #526082 passed

Pipeline passed for 4639a39a on graven:simplify-algorithm-constructor

Merged by Marco ClemencicMarco Clemencic 6 years ago (Jan 15, 2019 10:20am UTC)

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • @graven : could you please rebase?

    or should this be marked WIP until !577 (merged) or !765 is done?

  • I'd prefer to get !703 (merged) merged first, and fix the conflicts here afterwards (because there will be new conflicts).

  • Author Developer

    :thumbsup: -- I agree with Marco on this one.

  • @graven, can you fix the conflicts, please?

  • Marco Clemencic changed milestone to %v31r0

    changed milestone to %v31r0

  • Marco Clemencic resolved all discussions

    resolved all discussions

  • Marco Clemencic approved this merge request

    approved this merge request

  • @graven, I fixed the conflicts in https://gitlab.cern.ch/gaudi/Gaudi/tree/mr766-update

    If it's OK with you, you can update your branch or I can merge by hand.

  • Marco Clemencic mentioned in commit 2ffb70a5

    mentioned in commit 2ffb70a5

  • mentioned in merge request atlas/athena!20414 (merged)

  • Please register or sign in to reply
    Loading