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

Readme 404 fix #209

Open
wants to merge 34 commits into
base: master
Choose a base branch
from

Conversation

missing-user
Copy link
Contributor

@missing-user missing-user commented Nov 15, 2024

Suggested fix for #205 the broken URLs in the readme.

  • Added the citations file
  • Generally cleaned up the README a bit, added headings and the SPEC logo
  • Also made minor fixes related to the doxygen grouping, where some routines that produce hidden files weren't part of grp_output , were lacking doxygen params, etc...

@smiet
Copy link
Collaborator

smiet commented Nov 23, 2024

@smiet
Copy link
Collaborator

smiet commented Dec 1, 2024

Sorry, not familiar with .cff file handling, but the actions are giving an error:

 ruamel.yaml.scanner.ScannerError: mapping values are not allowed in this context
  in "<unicode string>", line 106, column 12

@missing-user
Copy link
Contributor Author

missing-user commented Dec 3, 2024

Sorry, not familiar with .cff file handling, but the actions are giving an error:

 ruamel.yaml.scanner.ScannerError: mapping values are not allowed in this context
  in "<unicode string>", line 106, column 12

I must have accidentally deleted a space before submitting. Should be fine now :)

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

Successfully merging this pull request may close these issues.

3 participants