search

sasjs / cli

Command line interface for creating, compiling, and building SAS® projects
https://cli.sasjs.io
MIT License
37 stars 5 forks source link

sasjs doc - UNC and lineage issues #757

Closed allanbowe closed 3 years ago

allanbowe commented 3 years ago

There are two issues with sasjs doc:

1) UNC paths not supported.

This is actually a doxygen limitation on windows, however to avoid confusion, we should identify such paths (eg by the fact they start with \\) and notify the user as follows:

UNC paths are not supported. Please map to a network drive, or migrate the project to an existing path (with a drive letter).

2) Lineage

Currently the lineage is not being picked up when the doxygen header is as follows:

/* 
  just one asterisk
*/

This approach works fine for sasjs compile, so it should work also for sasjs doc / lineage generation.

saadjutt01 commented 3 years ago

Since, it's doxygen's limitation to use comment block with double asterisks ( requires to start with double asterisks ) Reference: https://www.doxygen.nl/manual/docblocks.html#cppblock

Need to update lint for rule: hasDoxygenHeader

ghost commented 3 years ago

:tada: This issue has been resolved in version 2.38.8 :tada:

The release is available on:

Your semantic-release bot :package::rocket: