-
Notifications
You must be signed in to change notification settings - Fork 4
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
TS-31191 Add functionality to upload traces to multiple projects and via embedded resources #182
Conversation
embedded resources
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Very well done, this did not look like an easy change to make! 😁 I haven't tested the JSON-variant yet as I didn't quite get the use case, but let me know if I should take a closer look. 👀
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the great work again! 😊 I added just a few tiny comments.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome, thanks for all the fixes! 🎉 I saw one weird PowerShell anomaly, afterwards we can merge 😁
|
||
# Update the .resx file | ||
Update-ResxFile -resxPath $path -key $key -value $value -project $project | ||
if ($project -eq $null){ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah, sorry, I didn't notice this in the first round, but it looks like PowerShell actually gives a default value if you don't provide one for the parameter 😂 Can you modify the project
null checks either to compare with empty string, or as with the other parameters, do if (-not $project)
and if ($project)
? 😁
embedded resources
Addresses issue TS-31191
CHANGELOG.md
updatedPlease respect the Teamscale vote. Clean up findings before sending your change into review. Blacklist and report false-positives. Adjust the analysis profile if needed.