Consider moving stdio-socket to optional dependency

Open Beginner friendly
#345 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
2/5
Estimated time
1-3 hours
Newbie friendliness
65/100
Issue type
Refactor
Clarity
Mostly clear
Activity status
Quiet
Tech stack
python
Domain
build-system

Research direction

Start by locating the project’s dependency configuration and checking where stdio-socket is declared and whether the codebase imports it. Move it to an optional extra while keeping the base package installable on Windows, then verify dependency metadata and the existing test suite.

Written by the indexing model from the issue text.

Description

As best as I can tell this package is not used in the codebase, and is instead meant as a helper utility to work alongside fastcs. As such, I'd consider moving it into an extra, especially since it is a package that doesn't support windows hosts, since otherwise fastcs seems like it should work on any host architecture.

Dominant language
Python
Stars
6
Forks
8
Avg merge
2d 12h
Merged PRs (30d)
3

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 DiamondLightSource/fastcs

All issues in DiamondLightSource/fastcs

Similar issues

More Python issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.