Skip to content

Fix examples of repos CSV files#3299

Merged
MoralCode merged 1 commit intoaugurlabs:mainfrom
giordano:mg/repos-schema
Oct 14, 2025
Merged

Fix examples of repos CSV files#3299
MoralCode merged 1 commit intoaugurlabs:mainfrom
giordano:mg/repos-schema

Conversation

@giordano
Copy link
Copy Markdown
Contributor

@giordano giordano commented Oct 6, 2025

The code which parses the repos files interprets the first column as URL and the second one as repo group ID, but all the examples were doing the opposite.

Signed commits

  • Yes, I signed my commits.

The code which parses the repos files interprets the first column as URL and the
second one as repo group ID, but all the examples were doing the opposite.

Signed-off-by: Mosè Giordano <m.giordano@ucl.ac.uk>
@giordano giordano requested a review from sgoggins as a code owner October 6, 2025 20:47
Copy link
Copy Markdown
Collaborator

@sgoggins sgoggins left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

Copy link
Copy Markdown
Collaborator

@MoralCode MoralCode left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. While i'd rather figure out how to be more dynamic about this (such as including a header in the CSV format and using DictReader), this seems simple enough for now.

@MoralCode
Copy link
Copy Markdown
Collaborator

This has two approvals so im going to merge it

@MoralCode MoralCode merged commit 3fb2bae into augurlabs:main Oct 14, 2025
10 checks passed
@giordano giordano deleted the mg/repos-schema branch October 14, 2025 20:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants