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

https://github.com/metanorma/isodoc/issues/630: Presentation XML refa… #602

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

opoudjis
Copy link
Contributor

@opoudjis opoudjis commented Jan 5, 2025

…ctor, terms

Metanorma PR checklist

@opoudjis opoudjis self-assigned this Jan 5, 2025
desgn.children = l10n "#{to_xml desgn.children}:"
out = desgn.parent
.at(ns("./fmt-preferred//semx[@element = 'preferred'][last()]"))
out or return
Copy link

Choose a reason for hiding this comment

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

Inconsistent indentation detected.

out.div **attr_code(id: node["id"]) do |div|
termdef_parse1(node, div, defn, source)
node.children.each do |n|
next if %w(preferred definition termsource fmt-title
next if %w(fmt-preferred fmt-definition fmt-termsource fmt-title
Copy link

Choose a reason for hiding this comment

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

Avoid immutable Array literals in loops. It is better to extract it into a local variable or a constant.

</references>
</sections>
<bibliography>

Copy link

Choose a reason for hiding this comment

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

Trailing whitespace detected.


</bibliography>
</itu-standard>
<itu-standard xmlns="http://riboseinc.com/isoxml" type="presentation">
Copy link

Choose a reason for hiding this comment

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

Use 2 spaces for indentation in a heredoc.

</terms>
</sections>
</itu-standard>
<itu-standard xmlns="http://riboseinc.com/isoxml" type="presentation">
Copy link

Choose a reason for hiding this comment

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

Use 2 spaces for indentation in a heredoc.

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.

1 participant