Update PandasExcelWriter to use pd.ExcelWriter

[noahridge]

Created to fix issue #946 -- PandasExcelWriter overwrites file

Changes

Changes to pandas_extensions.py in PandasExcelWriter to implement the pd.ExcelWriter and add appropriate keyword arguments to support this.

How I tested this

Locally via existing unit tests

Notes

Checklist

• PR has an informative and human-readable title (this will be pulled into the release notes)
• Changes are limited to a single goal (no scope creep)
• Code passed the pre-commit check & code is left cleaner/nicer than when first encountered.
• Any change in functionality is tested
• New functions are documented (with a description, list of inputs, and expected output)
• Placeholder code is flagged / future TODOs are captured in comments
• Project documentation has been updated if adding/changing functionality.

[skrawcz]

Looking good. Mind adding a link to https://pandas.pydata.org/docs/reference/api/pandas.ExcelWriter.html on line 1484.
Otherwise anything else to be done? It doesn't appear you need a specific unit test for this right?

[noahridge]

Thanks, I added the link in the docstring for PandasExcelWriter. Otherwise, I think everything should be ready.

There is an existing unit test under tests/plugins/test_pandas_extensions.py which tests the PandasExcelWriter and .save_data() method, so I don't think anything needs to be added here.

[skrawcz]

thanks @noahridge !