search

qooxdoo / qxl.apiviewer

qooxdoo apiviewer app
https://qooxdoo.org/qxl.apiviewer
MIT License
5 stars 5 forks source link

Add "qxl.apiviewer.include" environment variable #38

Closed cboulanger closed 4 years ago

cboulanger commented 4 years ago

It would be good to have a counterpart to "excludeFromAPIViewer" (which should be renamed to "qxl.apiviewer.exclude", see #36) named "qxl.apiviewer.include" to be able to create an API for a part of a namespace, like so:

{
  "environment": {
    "qxl.apiviewer.exclude" : ["qx.*"],
    "qxl.apiviewer.include" : ["qx.io.*"]
  }
}
hkollmann commented 4 years ago

There is a counterpart: "includeToAPIViewer": []

cboulanger commented 4 years ago

I tried this, but it didn't work (I must have done something wrong). But what do you think about renaming both environment variables (in a BC way) - much easier to read & remember and clearly associated with the package.

hkollmann commented 4 years ago

renaming is ok.

cboulanger commented 4 years ago

Ok, see #36. Closing.