unioslo / unioslo/mreg-cli

Støtte at et argument kan være en liste

Open
#30 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

bug enhancement
Dominant language
Python
Stars
2
Forks
7
Avg merge
6d 11h
Merged PRs (30d)
2

Description

Typisk i dag bruker man:
>> host info (ns1 ns2)
for å gjøre host info ns1 og host info ns2. Dette støtter ikke mreg-cli.
Tilsvarende kan man gjøre:
>> host add (foo bar) 10.0.0.0/
for å legge til to hoster samtidig. Følgelig må man skjønne når noe er en liste og når det er i eksempelet over henholdvis andre og tredje argument.

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.

Research direction

No files or tests are named. Start by tracing mreg-cli's command parsing for the host info and host add entry points, then compare how positional arguments are currently handled. Done means the documented parenthesized forms expand into the intended host operations without confusing the other positional arguments.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
cli
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.