add argparse for GH exporter by mandylan75 · Pull Request #61 · Imageomics/repo-exporter

mandylan75 · 2026-06-26T23:34:36Z

Copilot

Pull request overview

Adds an argparse-based CLI to gh_repo_exporter.py to support overriding .env configuration via command-line flags, aligning with Issue #57’s request to introduce CLI options before packaging.

Changes:

Introduces argparse CLI flags for token/org/spreadsheet/sheet/credentials/repo-type inputs.
Refactors update_google_sheet to accept spreadsheet_id, sheet_name, and creds_path as parameters instead of relying on globals/env-only reads.
Updates organization/repository fetching and sheet access to use the parsed CLI values.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

egrace479

This is mainly centered around the OS/Argparse interactions and some reordering for clarity.

Co-authored-by: Elizabeth Campolongo <38985481+egrace479@users.noreply.github.com>

Copilot

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

…nv var

egrace479

Since copilot is so worried about the token string

Co-authored-by: Elizabeth Campolongo <38985481+egrace479@users.noreply.github.com>

egrace479

LGTM! Changes in argument parsing can be added to #62.

add argparse for GH exporter

44ef432

mandylan75 requested review from Copilot and egrace479 June 26, 2026 23:34

Copilot started reviewing on behalf of mandylan75 June 26, 2026 23:41 View session

Copilot AI reviewed Jun 26, 2026

View reviewed changes

Comment thread gh_repo_exporter.py Outdated

Comment thread gh_repo_exporter.py Outdated

add argparse for HF exporter

a198d38

mandylan75 requested a review from Copilot June 30, 2026 16:59

Copilot started reviewing on behalf of mandylan75 June 30, 2026 17:00 View session

Copilot AI reviewed Jun 30, 2026

View reviewed changes

Comment thread gh_repo_exporter.py Outdated

Comment thread gh_repo_exporter.py Outdated

Comment thread hf_repo_exporter.py Outdated

Comment thread hf_repo_exporter.py Outdated

Apply suggestions from code review

d3b6e08

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

egrace479 requested changes Jul 2, 2026

View reviewed changes

mandylan75 and others added 2 commits July 2, 2026 12:26

Apply suggestions from code review

b7c55f0

Co-authored-by: Elizabeth Campolongo <38985481+egrace479@users.noreply.github.com>

update env reads for hf_exporter

02db0fd

egrace479 reviewed Jul 2, 2026

View reviewed changes

Comment thread hf_repo_exporter.py Outdated

egrace479 reviewed Jul 2, 2026

View reviewed changes

Comment thread hf_repo_exporter.py Outdated

mandylan75 and others added 2 commits July 2, 2026 15:00

Update hf_repo_exporter.py

8b153d3

Co-authored-by: Elizabeth Campolongo <38985481+egrace479@users.noreply.github.com>

delete repo type in hf-exporter

1058493

mandylan75 mentioned this pull request Jul 2, 2026

Add optional repo-type filter to hf_repo_exporter.py #64

Open

mandylan75 requested a review from egrace479 July 2, 2026 19:36

egrace479 requested a review from Copilot July 2, 2026 19:36

Copilot started reviewing on behalf of egrace479 July 2, 2026 19:37 View session

Copilot AI reviewed Jul 2, 2026

View reviewed changes

Comment thread gh_repo_exporter.py

Comment thread gh_repo_exporter.py Outdated

Comment thread hf_repo_exporter.py Outdated

Copilot started work on behalf of egrace479 July 2, 2026 20:32 View session

Update workflow to pass --repo-type as CLI arg instead of REPO_TYPE e…

57f50c0

…nv var

Copilot finished work on behalf of egrace479 July 2, 2026 20:33

Copilot started work on behalf of egrace479 July 2, 2026 20:34 View session

Copilot finished work on behalf of egrace479 July 2, 2026 20:35

egrace479 reviewed Jul 2, 2026

View reviewed changes

Comment thread gh_repo_exporter.py Outdated

Comment thread hf_repo_exporter.py Outdated

Comment thread hf_repo_exporter.py Outdated

Apply suggestions from code review

b200141

Co-authored-by: Elizabeth Campolongo <38985481+egrace479@users.noreply.github.com>

egrace479 mentioned this pull request Jul 2, 2026

refactor/ create architecture for package #62

Open

mandylan75 added 3 commits July 2, 2026 18:20

implement copilots org_name fix

f1789f7

update integration test to pass org_name explicitly

0dae716

revert to old token line

1674da0

mandylan75 requested a review from egrace479 July 2, 2026 22:38

egrace479 approved these changes Jul 2, 2026

View reviewed changes

mandylan75 merged commit ff0762b into main Jul 2, 2026
4 checks passed

mandylan75 deleted the feature/issue-57/add-CLI branch July 2, 2026 23:06

egrace479 mentioned this pull request Jul 10, 2026

Add CLI use option #57

Closed

Uh oh!

Conversation

mandylan75 commented Jun 26, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

egrace479 left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

egrace479 left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

egrace479 left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants