Skip to content

wrapper has no run_redlines method #2

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

Closed
regstuff opened this issue Feb 26, 2024 · 4 comments
Closed

wrapper has no run_redlines method #2

regstuff opened this issue Feb 26, 2024 · 4 comments
Labels
bug Something isn't working

Comments

@regstuff
Copy link

Kudos on a great project. This is one thing I've sorely missed in python - the ability to create track changed word docs.
However, using the python wrapper like so:
output = wrapper.run_redlines('AuthorTag', '/path/to/original.docx', '/path/to/modified.docx')
throws an error saying wrapper has no run_redlines method.

Running the redlines.exe in the bin folder using subprocess works just fine, so right now I'm utilizing that, but would be nice to use the python wrapper.

@JSv4
Copy link
Owner

JSv4 commented Mar 13, 2024

Thanks @regstuff. I'll try to take a look this week.

@JSv4 JSv4 added the bug Something isn't working label Mar 13, 2024
@JSv4
Copy link
Owner

JSv4 commented Mar 13, 2024

Oh, also, do you mind sharing the details of your system env?

@rishabh-sagar-20
Copy link
Contributor

Is that a typo in the documentation? Since I can see a method in the package with the name run_redline

@JSv4
Copy link
Owner

JSv4 commented Mar 31, 2024

Yeah, should just be a typo in the markdown documentation. Updated the MD docs.

@JSv4 JSv4 closed this as completed Mar 31, 2024
JSv4 pushed a commit that referenced this issue Apr 1, 2024
The code has been updated to include support for macOS in the build and extraction process of the project. A new section is added to perform build and compression for macOS, in addition to existing mechanisms for Windows and Linux. The extraction method in engines.py is also modified to handle macOS specific binaries.
JSv4 pushed a commit that referenced this issue Jun 24, 2024
Merge pull request #9 from ross-mcnairn-dev/main
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants