Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use Argument Clinic for csv #84853

hauntsaninja opened this issue May 19, 2020 · 2 comments

Use Argument Clinic for csv #84853

hauntsaninja opened this issue May 19, 2020 · 2 comments


Copy link

BPO 40676
Nosy @larryhastings, @hauntsaninja
  • gh-84853: Use Argument Clinic for csv (where possible) #20200
  • Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.

    Show more details

    GitHub fields:

    assignee = None
    closed_at = None
    created_at = <Date 2020-05-19.01:36:49.440>
    labels = ['expert-argument-clinic']
    title = 'Use Argument Clinic for csv'
    updated_at = <Date 2020-05-19.01:38:19.516>
    user = '' fields:

    activity = <Date 2020-05-19.01:38:19.516>
    actor = 'hauntsaninja'
    assignee = 'none'
    closed = False
    closed_date = None
    closer = None
    components = ['Argument Clinic']
    creation = <Date 2020-05-19.01:36:49.440>
    creator = 'hauntsaninja'
    dependencies = []
    files = []
    hgrepos = []
    issue_num = 40676
    keywords = ['patch']
    message_count = 1.0
    messages = ['369314']
    nosy_count = 2.0
    nosy_names = ['larry', 'hauntsaninja']
    pr_nums = ['20200']
    priority = 'normal'
    resolution = None
    stage = 'patch review'
    status = 'open'
    superseder = None
    type = None
    url = ''
    versions = []

    Copy link
    Contributor Author

    To do all the functions in csv, we'd need to have support for **kwargs, tracked in Am attaching a PR for the ones that can be made to work today.

    @ezio-melotti ezio-melotti transferred this issue from another repository Apr 10, 2022
    Copy link

    Closing as #20200 was merged, thanks @hauntsaninja! We can reopen to migrate the rest once AC supports more options.

    Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
    None yet

    No branches or pull requests

    2 participants