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

Librarian utility #104

Merged
merged 13 commits into from
Aug 9, 2021
Merged

Librarian utility #104

merged 13 commits into from
Aug 9, 2021

Conversation

dmundra
Copy link
Collaborator

@dmundra dmundra commented Aug 6, 2021

A lot of changes but I have summarized them here:

  • Renamed the catalog to match the order of the standards (wcag-2.0 then 508). The catalog is also now auto-generated with a librarian script using values from the catalog/data directory. I copied values from the manually created catalog to generate the data ones.
  • Ignoring markdowns in opat directory from prettier because it will keep reformatting the generated version.
  • Changes to the various README files, CLI documentation, and also added librarian documentation.
  • Regenerated the drupal-9.markdown example with the slight tweaks to the catalog. The catalog now has additions to make it easier to match some of the text expected in the output like 'Applicable Standards/Guidelines' and 'Chapter/Table' differences (update the schema to facilitate that).
  • Updated version of js-yaml and added script commands to help generate some of the markdown files without needing to remember the whole command.
  • Updated tests and error handling.

Closes: #77

Need follow-up tickets:

@dmundra dmundra requested review from mgifford and grugnog August 6, 2021 21:43
@dmundra dmundra self-assigned this Aug 6, 2021
Copy link
Collaborator

@mgifford mgifford left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good to me. Thanks Daniel.

@dmundra dmundra merged commit 802ff2e into main Aug 9, 2021
@dmundra dmundra deleted the librarian-utility branch August 9, 2021 13:35
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.

Create Librarian utility that can combine catalogs
2 participants