Skip to content

Update text model#70

Open
andrew2net wants to merge 1 commit intomainfrom
text-model
Open

Update text model#70
andrew2net wants to merge 1 commit intomainfrom
text-model

Conversation

@andrew2net
Copy link
Copy Markdown
Contributor

from = fdate&.text || ob_date(doc)
[{ owner: [{ name: name, abbreviation: abbreviation, url: url }],
from: from }]
org = RelatonBib::Organization.new name: name, abbreviation: abbreviation, url: url
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Line is too long. [91/80]

return [] unless t

RelatonBib::TypedTitleString.from_string t.text, "en", "Latn"
RelatonBib::TypedTitleString.from_string t.text, lang: "en", script: "Latn"
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Line is too long. [83/80]

language: "en",
script: "Latn",
}]
[RelatonBib::Abstract.new(content: content, language: "en", script: "Latn",)]
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Line is too long. [85/80]
Avoid comma after the last parameter of a method call, unless each item is on its own line.

# @param com [Nokogiri::XML::Element]
# @return [RelatonItu::ItuGroup::Period]
def itugroupperiod(period)
return until period
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

This loop will have at most one iteration.

VCR.use_cassette "code" do
results = RelatonItu::ItuBibliography.get("ITU-T L.163", nil, {}).to_xml
expect(results).to include %(<bibitem id="ITU-TL.163" type="standard" schema-version="v1.2.8">)
expect(results).to include %(<bibitem id="ITU-TL.163" type="standard" schema-version="v1.3.1">)
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Line is too long. [101/80]

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