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

Add eofs desafio3 to master #1018

Closed
wants to merge 19 commits into from
Closed

Conversation

erickofs
Copy link
Contributor

No description provided.

changes to match linter
changed unction to work with set, custo limit and prime quantity range added, printing one result by line added
Readme created on oot directory
Updates on the function preserving only calculations. Functions variables are now global. Print logic changed to avoid errors.
Readme updated to match new file's name
Update on error handling for constants to include a greater range of user's input scenarios
Sieve of Eratosthenes module created to allow escalability. Primos.py now only contains the function call within the loop.
Calculation module change to be inline with Python PEEP-8
Main module name changed from primos to __main__.py to be inline with Python best practices
Changes made on Redme to better clarify file functionality based on last changes
gitignore created to prevent cache files to be commited
Update on main script
Class PrimeCalc's methods updated to avoid overwriting on main's execution. Main update for proper execution
remove trail space from line 83.
Update on file docstring
@erickofs erickofs closed this Dec 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant