Skip to content
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

[WIP] Refactoring/dry #224

Closed
wants to merge 3 commits into from
Closed

Conversation

mondaini
Copy link

Short description of what this resolves: Fixing some DRY violations

Fixes: #223

@manparvesh
Copy link
Member

@mondaini I merged your other PR and it seems that your fork is not updated. Please update it and resolve the merge conflicts.

Copy link
Member

@manparvesh manparvesh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

update fork

@mondaini
Copy link
Author

No problem @manparvesh :-)

There are other DRY violations I found that I intend to fix. I believe that the project's Coverage will improve after these changes.

@codecov-io
Copy link

Codecov Report

Merging #224 into master will decrease coverage by 4.55%.
The diff coverage is 62.1%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #224      +/-   ##
==========================================
- Coverage   76.37%   71.81%   -4.56%     
==========================================
  Files          59       55       -4     
  Lines        3357     3151     -206     
==========================================
- Hits         2564     2263     -301     
- Misses        793      888      +95
Impacted Files Coverage Δ
resources/hackerearth/result.py 100% <ø> (ø) ⬆️
setup.py 100% <ø> (ø) ⬆️
resources/hackerearth/language.py 100% <ø> (ø) ⬆️
modules/__init__.py 100% <ø> (ø) ⬆️
tests/test_chat.py 100% <100%> (ø) ⬆️
tests/dev/test_whois.py 100% <100%> (ø) ⬆️
tests/test_weather.py 95.23% <100%> (ø) ⬆️
tests/test_asciiator.py 100% <100%> (ø) ⬆️
tests/test_goals.py 100% <100%> (ø) ⬆️
tests/life/test_rlist.py 100% <100%> (ø) ⬆️
... and 82 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6e8b3e4...1b24024. Read the comment docs.

click.echo("Number | Date ")
click.echo("--------------------")
for i, some_file in enumerate(range(0, len(list_of_files))):
click.echo(str(i) + " |" + list_of_files[some_file][0:10])
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

for i, some_file in enumerate(list_of_files):
    click.echo(str(i) + "      |" + some_file[:10])

click.echo("Number | Date ")
click.echo("--------------------")
for i, some_file in enumerate(range(0, len(list_of_files))):
click.echo(str(i) + " |" + list_of_files[some_file][0:10])
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same as above?

@manparvesh
Copy link
Member

v1 is now deprecated, closing old PRs

@manparvesh manparvesh closed this Oct 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants