-
Notifications
You must be signed in to change notification settings - Fork 95
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
aggregate documented projects from config-file, not awesome-dat README #79
aggregate documented projects from config-file, not awesome-dat README #79
Conversation
See failed build logs below:
|
@@ -6,7 +6,7 @@ | |||
"scripts": { | |||
"update:token": "npm run update:list && cat repos.txt | ecosystem-docs sync -t ${GH_TOKEN} && cat repos.txt | ecosystem-docs read -t ${GH_TOKEN} | node scripts/build.js", | |||
"update": "npm run update:list && npm run update:build", | |||
"update:list": "curl -Ssl https://raw.githubusercontent.com/datproject/awesome-dat/master/readme.md | grep '📔' | egrep -o 'github.com/[a-zA-Z0-9_-]+/[a-zA-Z0-9_-]+' | cut -f 2- -d / | uniq > repos.txt", | |||
"update:list": "curl -Ssl https://raw.githubusercontent.com/datproject/docs/master/readme.md | grep '📔' | egrep -o 'github.com/[a-zA-Z0-9_-]+/[a-zA-Z0-9_-]+' | cut -f 2- -d / | uniq > repos.txt", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@joehand Does this work? On my local test it did not because it has to be README.md
in capitals for the Docs project.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
no, I had to fix with another commit 15499fe
This PR is related to the overhaul of
awesome-dat
where the new awesome list no longer contains system markers (the folder icons). See: dat-ecosystem-archive/awesome-dat#4 (comment)The list of project README's to aggregate is simply taken in the same manner from
aggregated-docs.md
in the Docs project instead.