-
Notifications
You must be signed in to change notification settings - Fork 3
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
update_repo
errors
#308
Comments
|
@matentzn I just updated my ODK dev image again, thanks. I added a note about possibly needing to run However, there are two important things to note:
|
The linked PR does not contain config file changes. |
Hey @matentzn I think you may be misunderstanding, but I have comments related to this in a number of different places so it is a bit confusing. PR #307 is just for documentation. |
@matentzn Just noting that, (i) running it 2x produces no new results (context), (ii) but running it a second time does produce no errors), and (iii) I'm still getting errors from running it the first time (see copy/paste of log below; or here for context). Errors
Running with -Xmx20G of memory for ROBOT and Java-based pipeline steps. mondo-ingest.Makefile:70: warning: overriding recipe for target 'components/omim.owl' sh ../scripts/update_repo.sh INFO:root: Copying: /tools/templates/CONTRIBUTING.md.jinja2 -> target/mondo-ingest/CONTRIBUTING.md.jinja2 Repository files have been successfully copied, but no git commands have been run. total 0 WARNING: These files should be manually migrated: mkdocs.yaml, .gitignore, src/ontology/catalog.xml (if you added a new import or component) ../scripts/update_repo.sh: 39: Syntax error: Unterminated quoted string |
Suggestion: |
Hmm.. Not a bad idea actually. Can you make an ODK issue about this? I will see if I implement that! |
Overview
I ran
sh run.sh make update_repo
, and it appears to have worked correctly, at least partially. However, I got these errors at the end:Logs
log.txt
The text was updated successfully, but these errors were encountered: