-
Notifications
You must be signed in to change notification settings - Fork 247
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
Field of Science ID's #3802
Field of Science ID's #3802
Conversation
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.
OSG Staff:
There was an unexpected error verifying this PR - please review.
-- OSG-BOT 💥
Output from the automerge downtime check script:
Traceback (most recent call last):
File "/app/webapp/automerge_check.py", line 288, in <module>
sys.exit(main(sys.argv[1:]))
File "/app/webapp/automerge_check.py", line 127, in main
orgs_new = get_organizations_at_version(head)
File "/app/webapp/automerge_check.py", line 213, in get_organizations_at_version
return set( p.get("Organization") for p in projects )
File "/app/webapp/automerge_check.py", line 213, in <genexpr>
return set( p.get("Organization") for p in projects )
AttributeError: 'NoneType' object has no attribute 'get'
You need to quote these new values, especially the field of science IDs which are being interpreted as floating point numbers. Also projects/GRAPLEr.yaml at least is not syntactically correct -- looks like the original file didn't have a newline at the end, so appending stuff broke it. |
@matyasselmeci I realize this is a brutal review so let me know if you want anything double ( triple ) checked. Checks I have done:
|
Closing in favor of not solving 1100 conflicts -> #3809 |
No description provided.