-
Notifications
You must be signed in to change notification settings - Fork 19
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 resource gallery #468
Conversation
Correct the fact that json file data being dumped into yaml file with incorrect format
for more information, see https://pre-commit.ci
👋 Thanks for opening this PR! The Cookbook will be automatically built with GitHub Actions. To see the status of your deployment, click below. |
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.
This was much needed; thanks a lot for doing it!
Seems to still be having issues: https://github.com/ProjectPythia/projectpythia.github.io/actions/runs/12240686083/job/34144102576 |
It seems to be due to a validation issue this time though (previously validation was passed and PR was created but the PR's changed file was ill-formatted and mistaken content). The validation failure is probably due to a format problem in the issue you created. Please compare with this older issue and try to match its format/content |
@erogluorhan I actually copied that issue to create the test. But I'm realizing now that I copied the formatted content instead of the raw. |
Also that old issue is triggering validation errors now (I think because Michaela's affiliation isn't a string). I'm not sure why it is more picky now than it used to be. It would be nice if there was some sort of comment when validation checks fail. E: It was actually upset because one author had no affiliation information. Opening a new PR to fix the "open PR" failure in #475 |
Closes #439