CI tasks/tests shown independent of branch

Open
#2,356 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
4/5
Estimated time
3-5 days
Newbie friendliness
25/100
Issue type
Bug
Clarity
Needs clarification
Activity status
Stale
Tech stack
python
Domain
ci-cd

Research direction

Start by examining the CI behavior described for commit 6e0394f981cec9774aeac4919c2c7c7aea094aba when it is selected on the master and testing branches. Trace how runs are identified and displayed for fast-forwarded revisions. Done means the issue has a decided, tested behavior for whether previously completed tasks should be shown or rerun on another branch.

Written by the indexing model from the issue text.

Description

https://github.com/mozilla/code-review has https://github.com/mozilla/code-review/commit/6e0394f981cec9774aeac4919c2c7c7aea094aba as the head which also got pushed to the testing branch now. The CI runs again and the brown circle for pending or running CI tasks is shown both with the master and testing branches selected in Github but it already ran for master earlier.

  1. Does the CI run get identified only based on revision and not also on branch? Should this be changed
  2. If there has already been a CI run for the given revision and it gets pushed to a different branch as a fast-forward, should tasks be omitted from running again if they are unnecessary for deployment?
Dominant language
Python
Stars
67
Forks
52
Avg merge
3d 23h
Merged PRs (30d)
54

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from mozilla/code-review

All issues in mozilla/code-review

Similar issues

More Python issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.