Vscode-extension-for-zowe: Circular dependencies

Created on 14 Apr 2020  路  3Comments  路  Source: zowe/vscode-extension-for-zowe

Madge finds 28 circular dependencies on the master branch.

image (1)

Technical Debt

Most helpful comment

We should think about adding madge as part of the npm run build script too.
This will avoid having this problems in the future : )

All 3 comments

One of the main cyclic dependencies to address is Profiles => extension getZoweDir(). I would hope this function could be relocated simply and would address 11 of the entries in the table above.

One of the main cyclic dependencies to address is Profiles => extension getZoweDir(). I would hope this function could be relocated simply and would address 11 of the entries in the table above.

I will look into moving this function, if it hasn't been moved already once @zFernand0 / my extension refactoring PRs are merged...

We should think about adding madge as part of the npm run build script too.
This will avoid having this problems in the future : )

Was this page helpful?
0 / 5 - 0 ratings