-
Notifications
You must be signed in to change notification settings - Fork 77
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
Clean up more the output of convert_docx_to_markdown
#1068
Comments
Hi @samarth9008 Could you please provide access to the Gdoc file mentioned above |
@surbhi498 and @mayank922 Seeing the complexity of the task, can you guys pls co-ordinate and work on it together. It might take long for one person to work on it. Consider it as one of the team work testing task. |
Hi @samarth9008 Could you please provide access to the Gdoc file mentioned above |
Shared. |
Hi @samarth9008 I am unable to find |
To run the linter follow this guide |
Hi @samarth9008 I did follow the one in the docs for my previous PR but this doc We could correct this line
|
This was the old way of running linter specifically on md files. Lately we merged with our python linter and even md files can be lint using Feel free to create a PR for it. |
Hi @samarth9008 The second task would be done by you right? |
Yes |
Hi @samarth9008, |
Instead of testing the script, can we only tests the function used in the function. |
Do you mean that we just test the rest of the functionalities of this fun and don't test the docker command which uses the interactive mode?
|
Hi @samarth9008 How do you want us to test the _move_media( ) function? We can write a test case where if there is To test for a directory that exists do you want us to create a test directory? |
For now lets do one thing Try to find tests cases similar to this case. Test case about scripts and see how they are tested. Try to look around the code to understand how temp dir can be created. Looking through other code can give more idea. If function seems complex or have no idea how to move forward, lets leave a TODO and we will address it separately in a different issue. |
I think we can leave a TODO for _convert_docx_to_markdown function. We will try again working on |
I have raised PR for the Script. The Link is given as below #1092 (comment) |
Hi @samarth9008, |
The doc for the flow is here
docs/documentation_meta/all.gdocs.how_to_guide.md
The script is here
dev_scripts/documentation/convert_docx_to_markdown.py
Convert a Gdoc file into a markdown - https://docs.google.com/document/d/1vg2IipQ4csA18hv041uRjxyF_vcdHrWuhpejf2zhaUs/edit
Merge it and remove the Gdoc
Add unit tests for the script -
Extend the script to handle a few corner cases
I would focus on one thing at a time. Start with something easy and have multiple PRs to achieve the goal to ease up the review
FYI @gpsaggese @sonaalKant
The text was updated successfully, but these errors were encountered: