Sonya Nikiforova
Sonya Nikiforova
> I ran pytest before and after updating test_amp_doc_formatter.py with get_scratch_space(). Some of the test cases are failing in both cases. Just wanted to know if I need to debug...
Closing as done
> Operating System: Windows Unfortunately, our dev env is only adapted for Linux and Mac at the moment, see [docs](https://github.com/causify-ai/helpers/blob/master/docs/onboarding/intern.set_up_development_on_laptop.how_to_guide.md#supported-os). Will it be possible for you to install dual boot...
Correct, we don't use GH sub-issues at the moment. BTW, for the "final checks" part of the checklist it's better to do something very small but useful -- like fixing...
@Peeyush4 can you confirm that you've completed the checklist: the setup is working and you've read all the docs? In which case we'll close this issue and move on to...
> Should I proceed with the test fixes? Of course. In the future, go ahead and fix the failing checks before requesting review. Also, you don't have to provide a...
@aangelo9 pls also make sure to [resolve conversations](https://github.com/causify-ai/helpers/blob/master/docs/coding/all.submit_code_for_review.how_to_guide.md#resolve-conversations) after you've addressed them and keep the branch [up to date with master](https://github.com/causify-ai/helpers/blob/master/docs/coding/all.submit_code_for_review.how_to_guide.md#merge-master-into-your-branch) (you can use "Update branch" button on the PR...
- The empty line should be removed by the `amp_remove_empty_lines_in_function` step. We should add a test for it reproducing this use case and debug. - Not sure which step is...
Re documentation: we already have [this doc](https://github.com/causify-ai/helpers/blob/master/docs/work_tools/docker/all.dockerized_flow.explanation.md) on dockerized executables. It can be updated if needed.
> Sure. We can also have other tasks for (docs -> md) if we want tasks for interns. @sonniki LMK thanks Yes please, feel free to file issues (one per...