search

LiveRamp / reslang

A language for describing resource-oriented APIs & turning them into Swagger or resource diagrams. Oriented around the concepts we want to expose in the APIs.
Apache License 2.0
23 stars 7 forks source link

Namespaces containing a slash can't be imported. #123

Closed ChristianHansen closed 3 years ago

ChristianHansen commented 4 years ago

It appears that namespaces containing a / cannot be imported. See https://github.com/LiveRamp/api-specs/pull/325 for an example.

cjea commented 3 years ago

I think this is actually not a bug. import deals with peer directories, not with namespaces. The docs are misleading / wrong in calling them namespaces. But it might make sense that slashes aren't allowed if only peer directories are intended to be able to imported.

I don't see why that should be a restriction, but it does appear to be the design decision. So I imagine that there is no need to currently be importing non-peer directories, but if there becomes a need, this PR could be taken out of draft mode https://github.com/LiveRamp/reslang/pull/129/files