MetOffice / MetOffice/CMEW

Move duplicate `write_recipe`/`write_yaml` function out of tasks and import from single source

Open
#236 0 comments 0 reactions 0 assignees View on GitHub
configure enhancement technical debt
Dominant language
Python
Stars
5
Forks
3
Avg merge
5d 6h
Merged PRs (30d)
8

Description

The function `write_recipe` in `configure_standardise`/`update_recipe_file.py` duplicates the functionality of `write_yaml` in `configure_process`/`configure_process.py`.

To remove this duplication, this functionality should be moved to a suitable location so it can be imported into both tasks. @ehogan suggested Cylc offers a solution for this.

Contributor guide

Open the contributing guide

Research direction

Start by comparing write_recipe in configure_standardise/update_recipe_file.py with write_yaml in configure_process/configure_process.py, then review the referenced Cylc solution. Identify a suitable shared location and verify that both tasks import the single implementation without retaining duplicate functionality.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
tooling
Issue type
Refactor
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
38/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.