influxdata / influxdata/influxdb
Delete (or move) hand-written HTTP clients
- Dominant language
- Rust
- Stars
- 31.7k
- Forks
- 3.7k
- Avg merge
- 13h 37m
- Merged PRs (30d)
- 8
Description
#21797 removed the CLI code from this repo, but left the hand-written HTTP clients because they're heavily used by integration tests.
- [ ] Determine which clients are safe to delete, and which are used by tests
- [ ] Delete all clients that are safe
- [ ] Consolidate clients used for testing into a single package focused on testing
Item 3 will probably require significant refactoring, because the existing clients use package-private constants for things like HTTP prefixes. We could possible adapt the codegen system used by the new CLI repo to generate HTTP clients here.
Contributor guide
Research direction
Start by inventorying the hand-written HTTP clients and searching the integration tests for their use. Done means deleting clients that are safe to remove and consolidating test-used clients into one testing-focused package; the issue notes that this may require significant refactoring or code generation.
Written by the indexing model from the issue text.
Assessment
- Domain
- testing
- Issue type
- Refactor
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 25/100