MetOffice / MetOffice/CDDS

Make accessing the CddsPlugin nicer

Open
#534 0 comments 0 reactions 0 assignees View on GitHub
migrated project plugins
Dominant language
Python
Stars
8
Forks
7
Avg merge
1d 15h
Merged PRs (30d)
23

Description

Migrated Jira Issue CDDSO-167
Jira creation date 06/07/2022 10:35
-------
When the plugin is accessed, the instance of the ``PluginStore`` must be accessed and then the plugin:

```PluginStore.instance().get_plugin()```

Because now we only store one plugin instead of multiple, we could simplify it a bit by provide a ``classmethod`` ``property`` function:

```PluginStore.plugin```

This would simplify many calls and make the code more compact.
-------

Contributor guide

No contributing guide indexed for this repository

Research direction

Search for PluginStore and the existing PluginStore.instance().get_plugin() calls to identify the affected entry points. Replace the single-plugin access pattern with PluginStore.plugin and verify that all existing plugin access continues to work through the project's test suite.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
backend
Issue type
Refactor
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Clearly specified
Newbie friendliness
55/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.