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

Infer doesn't lowercase resource names #118

Open
pripley123 opened this issue Jun 6, 2022 · 1 comment
Open

Infer doesn't lowercase resource names #118

pripley123 opened this issue Jun 6, 2022 · 1 comment
Assignees
Labels

Comments

@pripley123
Copy link

pripley123 commented Jun 6, 2022

Overview

According to https://specs.frictionlessdata.io/data-resource/#metadata-properties it appears that resource name must be lowercase. When using the infer command it appears that the inspected value only removes the extension and does not lowercase the resource name (inspection.name = pathModule.basename(path[0], .${inspection.format}) (line 579 in src/resource.js). Hence MyFile.csv would have an inferred resource name of MyFile as opposed to myfile which produces invalid package error if you commit to it. Wondering if there's a reason no lower casing is occurring?


Please preserve this line to notify @aivuk (lead of this repository)

@aivuk aivuk added the bug label Jun 8, 2022
@aivuk aivuk self-assigned this Jun 8, 2022
@aivuk
Copy link

aivuk commented Jun 8, 2022

Thanks for reporting it, looks like a bug. As soon as I fix it I will ping you here.

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

No branches or pull requests

2 participants