add node auth token env in vulnerability-and-outdated-packages-report#42
add node auth token env in vulnerability-and-outdated-packages-report#42muhamedsk wants to merge 1 commit into
Conversation
9de1171 to
9cea6ea
Compare
|
hmm... could be tricky |
|
@muhamedsk seeing as @nasirky is very busy atm, it might be good, that you investigate a fix for that yourself. @nasirky do you disagree? |
|
@KlausNie this is blocking me from running the reports in BSI projects (and creating the TRDP reports). I am not sure, but seems like the solution to this should live in
|
I agree! As Nasir is on vacation, we'll have to find a solution. Or this needs to wait until Nasir is back. Either way, I won't have the time to find a solution to this in the next weeks. So my suggestion is, you look into it |
@KlausNie what exactly are your considerations? That env should be present only during that concrete step. That is only needed during yarn install. The actual Since next steps don't inherit |
I haven't looked into it. I just felt like it could be difficult to get that to work |
@KlausNie My idea is trying this out, by first merging, and then try running the dependency report workflow in:
If both work, then we should be fine! |
dependency report for web is failing due to the missing
NODE_AUTH_TOKENwhich is required for the internal QB npm packages used in the projects.The error:
More info here