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

Skohub-vocab does not build in fork repository but builds in cloned repo #177

Closed
dchoi127 opened this issue Mar 31, 2022 · 4 comments
Closed
Assignees

Comments

@dchoi127
Copy link

When running "npm run build" in a cloned repository the schema construction works as intended. However, when running "npm run build" in a forked repository the schema construction terminates with the error message below.

Screen Shot 2022-03-30 at 9 51 29 PM\

With the error lying in url-template.js, I used the diff command to observe if there were any differences between the url-template.js in the forked repository vs the cloned repository. Strangely enough, the diff command returned that there were no differences.

IMPORTANT NOTE: The files in src/common.js and the data directory were changed slightly. Changes seen in #176 . However, I do not see these changes causing a problem. Is there something I am overlooking?

The forked repository link: https://github.com/dchoi127/skohub-vocabs

@acka47
Copy link
Member

acka47 commented Mar 31, 2022

I can not reproduce this. It worked fine for me with node v16.13.1:

  1. Fork to https://github.com/acka47/skohub-vocabs
  2. $ git clone git@github.com:acka47/skohub-vocabs.git skohub-vocabs-forked-test
  3. $ cd skohub-vocabs-forked-test
  4. $ nvm use 16
  5. $ npm i
  6. $ cp .env.example .env
  7. $ cp test/data/systematik.ttl data/
  8. $ rm -rf .cache (which is probably unnecessary at this point)
  9. $ npm run build
  10. $ cd public/
  11. $ python3 -m http.server
  12. Open http://0.0.0.0:8000/w3id.org/class/hochschulfaecher/scheme.de.html in browser et voilà

image

@tombaker
Copy link

@acka47 The sequence above also works with node v17.8.0 and helminths.ttl (corrected).

@acka47
Copy link
Member

acka47 commented Apr 1, 2022

@dchoi127 Please close the issue if your problem is resolved.

@dchoi127 dchoi127 closed this as completed Apr 1, 2022
@dchoi127
Copy link
Author

dchoi127 commented Apr 6, 2022

Problem was resolved. Thank you!

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

No branches or pull requests

3 participants