Remove rexml as a runtime dependency

[ekohl]

This was added in 660a798 (see #871 as well), but I can't reproduce the failure. I also can't find out where rexml was used in the past.

[google-cla]

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

[ekohl]

Before I sign any CLA I would like to know if this is even valid, or if I'm just missing something where it is needed.

[bajajneha27]

Hi @ekohl
Help me understand, are you saying that you tried removing this gem and you couldn't reproduce the failure? Can you please also tell me how you tried to check for failures?

[ekohl]

I've tried to look at the git history to see why it was added. It clearly was added, likely because of some failure. However, now removing the dependency still passes the test suite so I'd think it's redundant. Having fewer dependencies is better so I'd like to see it dropped.

Does that clarify things?

[bajajneha27]

Thanks for clarifying!
Can you please sign the CLA for us to proceed with the PR?

[ekohl]

I would need to discuss that with legal. Is there a chance you could submit a similar PR with these changes instead?

[bajajneha27]

Sure. We'll try to cover this in a separate PR. Closing this one. Thank you!