Arpit Jain

Results 9 comments of Arpit Jain

Request review from @mxyng @dhiltgen @bmizerany @anaisbetts @Cephra @jmorganca (can't add to reviewer list on the PR)

@eliegoudout no I do not. I didn't know this repo is not being actively managed anymore. This repo is being used by a lot of devs (more than 5M downloads...

@astanin thank you for the reply. I see that there are some merge conflicts now - let me resolve them and send the PR back to you for review.

Apologies for the cold mention @ToniBonittoGSA @nick-mon1 @bonnieAcameron @clmedders @mhipkegsa -- Found you from top contributors the repo recently. This PR improves the code quality for the repo by fixing...

@havetisyan noted. Thank you for the clarification. I opened a PR because there is still code related to zpu_updater. The changes in the PR should then suffice as improving code...

> Removing the rexml entry altogether allows it to go to the latest with no issues. Should I point the PR to latest branch rather? Apologies, I got a bit...

> Sorry for that. Remove the entry and keep it pointed to sequoia. Thanks. Done

Please review this PR as well @golbiga if and when you have time. Thank you.

You can find patched security vulnerabilities in this fork repo - https://github.com/arpitjain099/palantir-python-sdk/pulls?q=is%3Apr+label%3Adependencies+is%3Aclosed It is important to merge this as this SDK gets used by other developers as well. **Type** *...