Update diff commands in UMDP3 fixer

Open
#208 1 comment 0 reactions 1 assignee View on GitHub

@Pierre-siddall is already working on this.

Since Mar 19, 2026.

Assessment

This issue has not been assessed yet.

Description

bug

UMDP3 fixer currently uses fcm diff and bdiff commands to take the difference between the equivalent of the main branch and the developers branch for example in this statement:

def get_branch_diff():
    """If in a local working copy of an FCM branch, return a
    list of files returned by running a branch-diff command"""

    # Use the bdiff command to extract a list of files that have changed
    # on the user's branch
    bdiff = subprocess.Popen(
        "fcm bdiff --summarize", stdout=subprocess.PIPE, shell=True
    )

    bdiff_stdout, _ = bdiff.communicate()
    bdiff_stdout = bdiff_stdout.decode(sys.stdout.encoding)

Therefore UMDP3 fixer needs to be refactored to reflect the move to GitHub.

Dominant language
Python
Stars
9
Forks
19
Avg merge
5d 48m
Merged PRs (30d)
4

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 MetOffice/SimSys_Scripts

All issues in MetOffice/SimSys_Scripts

Similar issues

More Python issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.