Skip to content

Schedule regular XML validation#8124

Open
adunning wants to merge 2 commits intocitation-style-language:masterfrom
adunning:xml-validation
Open

Schedule regular XML validation#8124
adunning wants to merge 2 commits intocitation-style-language:masterfrom
adunning:xml-validation

Conversation

@adunning
Copy link
Copy Markdown
Contributor

Augment the existing XML validation by creating inline annotations on pull requests and validating the entire repository weekly to ensure it is up-to-date with any improvements to the schema.

Copilot AI review requested due to automatic review settings March 31, 2026 18:29
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds a GitHub Actions workflow to validate CSL (XML) files on PRs/pushes and to run a full-repository validation on a weekly schedule, helping keep the repository aligned with the upstream CSL schema.

Changes:

  • Introduces a new Validate XML workflow triggered on PRs and pushes affecting **/*.csl.
  • Adds a weekly scheduled run (and manual dispatch) that validates the full repository.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@github-actions
Copy link
Copy Markdown

😟 There are some issues with your submission.

2 tests failed

dependent/adhd-attention-deficit-and-hyperactivity-disorders: must validate against the CSL 1.0.1 schema
Please check your style at http://validator.citationstyles.org/

expected `[[0, "ERROR: Element style failed to validate attributes"], [4, "4:0: ERROR: Expecting an element aut...ment info failed to validate content"], [2, "2:0: ERROR: Element style failed to validate content"]].empty?` to be truthy, got false

dependent/adhd-attention-deficit-and-hyperactivity-disorders: must be parsable as a CSL style

not allowed to add #<CSL::Info::Id "http://www.zotero.org/styles/adhd-attention-deficit-and-hyperactivity-disorders"> to #<struct CSL::Style::Children :"style-options"=nil, info=#<CSL::Info children=[2]>, locale=[], macro=[], citation=nil, bibliography=nil>

Please check the test report for details.

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.

2 participants