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

Suggestion: clarify Masked and Readonly Paths #509

Closed
Mashimiao opened this issue Jun 24, 2016 · 2 comments
Closed

Suggestion: clarify Masked and Readonly Paths #509

Mashimiao opened this issue Jun 24, 2016 · 2 comments

Comments

@Mashimiao
Copy link

There is not clearly requirement for Masked or Readonly Paths.
I think we must definitely define they must be absolute path or not in runtime spec.

@wking
Copy link
Contributor

wking commented Jun 24, 2016

On Fri, Jun 24, 2016 at 02:31:48AM -0700, Ma Shimiao wrote:

I think we must definitely define they must be absolute path or not
in runtime spec.

This is related to a general need to specify the anchor for relative
paths 1. If there is a config.md-wide entry like:

Relative paths are interpreted relative to the directory holding
config.json.

Then having relative paths here (or in other places) seems
sufficiently well-defined to me.

 and the next two lines.

@hqhq
Copy link
Contributor

hqhq commented Dec 24, 2016

Fixed by: #587

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

3 participants