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

use 'group' instead of 'wg', 'wgURI', 'wgPatentURI' #563

Merged
merged 3 commits into from
Aug 6, 2020

Conversation

himorin
Copy link
Contributor

@himorin himorin commented Jul 23, 2020

(for future use)
From recent respec update, replace 'wg', 'wgURI', 'wgPatentURI' with 'group':

group configuration option

With a new configuration option — “group”, you can specify a “shortname” for the WG/CG and let ReSpec figure out the details for you. The wg, wgId, wgURI, and wgPatentURI options are deprecated in favour of “group”.

'timed-text' is from list


Preview | Diff

Copy link
Contributor

@nigelmegitt nigelmegitt left a comment

Choose a reason for hiding this comment

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

I don't think this change generates the expected outcome. It has the effect of removing from the SoTD the sentence:

This document was published by the Timed Text Working Group as an Editor's Draft.

There's no other mention of the group that has made the document, other than informatively in the acknowledgments. I'm not sure if it passes pubrules either?

My expectation is that if this change is applied successfully there should be no diffs in the generated output document.

@himorin
Copy link
Contributor Author

himorin commented Jul 23, 2020

ah, yea. it'll be fixed in next site update by https://github.com/w3c/respec/issues/3032

also let me update on respec-w3c-common in this PR (to respec-w3c)

@himorin
Copy link
Contributor Author

himorin commented Aug 3, 2020

updated profile to respec-w3c from w3c-common, per https://github.com/w3c/respec/pull/2838

Copy link
Contributor

@nigelmegitt nigelmegitt left a comment

Choose a reason for hiding this comment

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

Thanks, that seems to fix it!

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