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

BUG: projects-update script not working #561

Closed
azeemshaikh38 opened this issue Jun 9, 2021 · 5 comments
Closed

BUG: projects-update script not working #561

azeemshaikh38 opened this issue Jun 9, 2021 · 5 comments
Labels
good first issue Good for newcomers kind/bug Something isn't working
Projects

Comments

@azeemshaikh38
Copy link
Contributor

azeemshaikh38 commented Jun 9, 2021

@asraa @naveensrinivasan - the projects-update script (https://github.com/ossf/scorecard/blob/main/cron/data/update/main.go) which adds dependencies to the projects.csv file doesn't seem to be working and throws a lot of errors when run. Is this expected? What's the current status of this script?

@azeemshaikh38 azeemshaikh38 added the kind/bug Something isn't working label Jun 9, 2021
@asraa
Copy link
Contributor

asraa commented Jun 9, 2021

Do you have logs? Dont' know the status but I can help diagnose

@azeemshaikh38
Copy link
Contributor Author

You can run this at HEAD to reproduce:

make build-update-script && ./cron/data/update/projects-update ./cron/data/projects.csv

It's dumping out various errors like these:

2021/06/09 14:21:18 unable to parse the vanity URL bitbucket.org/creachadair/shell parsing bitbucket.org/creachadair/shell: XML syntax error on line 11: expected attribute name in element
2021/06/09 14:21:26 unable to parse the vanity URL git.apache.org/thrift.git parse https://git.apache.org/thrift.git?go-get=1: no go-import meta tags
2021/06/09 14:21:41 unable to parse the vanity URL  import path doesn't contain a hostname
2021/06/09 14:21:44 unable to parse the vanity URL bitbucket.org/creachadair/shell parsing bitbucket.org/creachadair/shell: XML syntax error on line 11: expected attribute name in element
2021/06/09 14:21:49 unable to parse the vanity URL git.apache.org/thrift.git parse https://git.apache.org/thrift.git?go-get=1: no go-import meta tags

@asraa
Copy link
Contributor

asraa commented Jun 10, 2021

:( I'm not sure why they're assumed to be vanity URLs if they're not github. @naveensrinivasan

@naveensrinivasan
Copy link
Member

Some go repos aren't GitHub https://github.com/kubernetes/kubernetes/blob/master/go.mod#L12

@azeemshaikh38 azeemshaikh38 added the good first issue Good for newcomers label Jun 29, 2021
@justaugustus justaugustus added this to Backlog in Scorecard Feb 22, 2022
@spencerschrock
Copy link
Contributor

no longer needed as script was removed in #3269

@spencerschrock spencerschrock closed this as not planned Won't fix, can't repro, duplicate, stale Nov 9, 2023
Scorecard automation moved this from Backlog to Done Nov 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers kind/bug Something isn't working
Projects
Development

No branches or pull requests

4 participants