search

abhinavsp0730 / khronos

8 stars 3 forks source link

Fix: used the os module to prompt a directory change for accessing the README.md file #9

Closed Paul-Annay closed 1 year ago

Paul-Annay commented 1 year ago

What changes does this PR introduce?

A single line of code is added 

os.chdir('../')

This changes the current directory to parent, therefore, making the README.md file accessible while preserving the current files execution.

Fix #5

abhinavsp0730 commented 1 year ago

Hey, would you please run python -m build and upload the package using twine to test pypi? And see if the long description is rendering correctly?

Paul-Annay commented 1 year ago

Okay on it!

abhinavsp0730 commented 1 year ago

Make sure you upload it to testPyPi and not the actual PyPi. https://packaging.python.org/en/latest/guides/using-testpypi/

Paul-Annay commented 1 year ago

Yes, I have uploaded to the testPyPi package manager. Apparently I am having trouble in figuring out how to check whether the long_description is rendering correctly. Do I have to create a django-project and install django-khronos in order to execute any commands from the CLI?

Paul-Annay commented 1 year ago

long-description-correctly-rendered

Executing the command from the terminal shows the expected results

Paul-Annay commented 1 year ago

I have made slight modification to the code by replacing the open function with the with context manager and used raw file path instead of importing the os module.

abhinavsp0730 commented 1 year ago

Yes, I have uploaded to the testPyPi package manager. Apparently I am having trouble in figuring out how to check whether the long_description is rendering correctly. Do I have to create a django-project and install django-khronos in order to execute any commands from the CLI?

Can you send me the link?

Paul-Annay commented 1 year ago

Yeah, sure link

abhinavsp0730 commented 1 year ago

Yeah, sure link

Any idea why the text is not properly formatted?


Feautres:

1) Benchmark Report: Create benchmark report in the terminal whenever you run your django tests and shows top 10(this no. can be configured) slowest tests.

2) Parallel Test Execution: Works with --parallel out of the box.

3) Google Sheets Integration: Whole tests benchmark can be automatically uploaded to Google Sheets.

4) CSV Export: Whole tests benchmark can be saved in a CSV.

See here https://pypi.org/project/django-khronos/ it's properly formatted?

Paul-Annay commented 1 year ago

@abhinavsp0730 No I am a bit clue-less myself.

abhinavsp0730 commented 1 year ago

Okay gonna merge anyway.
Thanks for the contribution.