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

Strip webappsec- from filenames #142

Closed
foolip opened this issue Feb 14, 2019 · 7 comments
Closed

Strip webappsec- from filenames #142

foolip opened this issue Feb 14, 2019 · 7 comments

Comments

@foolip
Copy link
Member

foolip commented Feb 14, 2019

w3c/webref@cc0a43d renamed feature-policy.idl to webappsec-feature-policy.idl based on the new spec URL.

However, it seems a virtual certainty that the eventual shortname will be feature-policy, because the URL if any will be http://w3.org/TR/feature-policy/ and not http://w3.org/TR/webappsec-feature-policy/.

Flip-flopping the name back and forth would be frustrating, and there aren't any other files in reffy-reports with this prefix.

My suggestion, therefore, is to strip webappsec- as a prefix.

@foolip
Copy link
Member Author

foolip commented Feb 14, 2019

@tidoust

foolip referenced this issue in w3c/webref Feb 14, 2019
The GitHub repository and URL of the Feature Policy spec changed to:
https://w3c.github.io/webappsec-feature-policy/

Reffy creates a `webappsec-feature-policy.idl` file as a result.
@foolip
Copy link
Member Author

foolip commented Feb 25, 2019

@dontcallmedom do you think this would make sense?

@dontcallmedom
Copy link
Member

absolutely; can you submit a pull request to that effect? if not, I'll try to look into it later

@foolip
Copy link
Member Author

foolip commented Feb 26, 2019

At what level should this be done? When determining a shortname from URL, or when making a filename from the shortname? Code pointers? :)

@dontcallmedom
Copy link
Member

@foolip
Copy link
Member Author

foolip commented Feb 27, 2019

Do you know if the if (spec.shortname) branch will be taken, or if this needs to go inside if (githubMatch)?

@dontcallmedom
Copy link
Member

Ì think the one you care about is githubMatch - spec.shortname addresses specs that have already hit TR and have thus the shortname TR/foo. IIUC, the case we're dealing with is specs that haven't reached FPWD yet, and thus are only known by their editors draft github URL.

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

No branches or pull requests

2 participants