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

It's unclear that primitive type "uri" enclose URNs (not only URLs) #126

Closed
Kroisse opened this Issue Mar 23, 2017 · 3 comments

Comments

3 participants
@Kroisse
Contributor

Kroisse commented Mar 23, 2017

Currently, Nirum specification doesn't clarify their implementors to treat "uri" be a union of Uniform Resource Locator and Uniform Resource Name, or accept URL only and reject URN. Because URN is used rarely in practice, most of Nirum users might want to exclude URN during serialization and deserialization. Furthermore, most of programming languages don't have URN datatype in their standard library. It makes hard to implement the runtime for them.

@dahlia

This comment has been minimized.

Member

dahlia commented Mar 23, 2017

I suggest to rename uri to url.

@dahlia dahlia added the typ:refactor label Mar 23, 2017

@dahlia dahlia added this to the Version 0.5.0 milestone May 5, 2018

@dahlia dahlia added the dfct:easy label May 5, 2018

@Jhuni0123 Jhuni0123 self-assigned this May 21, 2018

Jhuni0123 added a commit to Jhuni0123/nirum that referenced this issue May 21, 2018

Jhuni0123 added a commit to Jhuni0123/nirum that referenced this issue May 21, 2018

Jhuni0123 added a commit to Jhuni0123/nirum that referenced this issue May 21, 2018

Jhuni0123 added a commit to Jhuni0123/nirum that referenced this issue May 21, 2018

@dahlia

This comment has been minimized.

Member

dahlia commented May 21, 2018

The compiler 0.4 will be released with a new type name url and deprecated uri (which is aliased to url).

In the next minor release we need to obsolete uri.

Jhuni0123 added a commit to Jhuni0123/nirum that referenced this issue May 25, 2018

@dahlia

This comment has been minimized.

Member

dahlia commented Aug 17, 2018

PR #281 implemented it.

@dahlia dahlia closed this Aug 17, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment