Skip to content

Fix examples of repos CSV files#3299

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

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

Conversation

@giordano
Copy link
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
Member

@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
Contributor

@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
Contributor

This has two approvals so im going to merge it

@MoralCode MoralCode merged commit 3fb2bae into chaoss: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