Lilian Kasem
Lilian Kasem
pinging @brettsam & @mathewc for a second pass!
> pinging @brettsam & @mathewc for a second pass! bumping again :)
Please update the PR name so it's more clear on what the change is, and update description to describe the changes that are being made - thanks!
@Tonewall is #8803 replacing this PR? Is so, please close it.
@pragnagopa is this something we still need, can you take another look?
> @pragnagopa @liliankasem Do you have insights on the impact of the change requested here? If it's closed, I would assume that we can remove that todo item, but @pragnagopa...
> We also have a `LICENSE_APACHE.txt` in the repo root. Is that to remain? Ah good point, @fabiocav can we remove that file do you think?
Looks like this is a known issue with a bug fix completed, it will be in the next release but there isn't an ETA on that right now. This is...
Tagging @fabiocav & @mathewc for a second pass, I think your comments were addressed?
Dependency changes here are minor, would this be okay? @fabiocav @brettsam ``` Changed: - Microsoft.IdentityModel.Abstractions.dll: 6.21.0.0/6.21.0.30701 -> 6.22.0.0/6.22.0.30727 - Microsoft.IdentityModel.JsonWebTokens.dll: 6.21.0.0/6.21.0.30701 -> 6.22.0.0/6.22.0.30727 - Microsoft.IdentityModel.Logging.dll: 6.21.0.0/6.21.0.30701 -> 6.22.0.0/6.22.0.30727 - Microsoft.IdentityModel.Tokens.dll:...