Skip to content
Snippets Groups Projects

Skip per-commit validations which have already passed on another change/branch

Merged Francisco Javier López requested to merge fj-44679-skip-per-commit-validations into master
All threads resolved!

What does this MR do?

Caches the result of validations per commit in order to avoid performing them if the commit is found in a different branch.

What are the relevant issue numbers?

Closes #44679 (closed)

Does this MR meet the acceptance criteria?

Edited by Francisco Javier López

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
  • added 1 commit

    Compare with previous version

  • Douwe Maan resolved all discussions

    resolved all discussions

  • Douwe Maan
  • Douwe Maan
  • Francisco Javier López marked the checklist item Changelog entry added, if necessary as completed

    marked the checklist item Changelog entry added, if necessary as completed

  • marked the checklist item Tests added for this feature/bug as completed

  • James Edwards-Jones
  • added 1 commit

    • 9816f409 - Code review comments applied

    Compare with previous version

  • added 1 commit

    • 3ba974e8 - Added caching for Change::Access diff validations

    Compare with previous version

  • added 1 commit

    • 87522992 - Added caching for Change::Access diff validations

    Compare with previous version

  • @DouweM can you take another look? The only job failing is danger-review and was fixed in the last push.

  • Douwe Maan
  • added 1 commit

    • 10921133 - Renamed process_raw_deltas to process_commits

    Compare with previous version

  • Douwe Maan resolved all discussions

    resolved all discussions

  • Douwe Maan approved this merge request

    approved this merge request

  • Douwe Maan enabled an automatic merge when the pipeline for 10921133 succeeds

    enabled an automatic merge when the pipeline for 10921133 succeeds

  • merged

  • Douwe Maan mentioned in commit 08f86409

    mentioned in commit 08f86409

  • Please register or sign in to reply
    Loading