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

Improved Documentation: Added comments in application/worker.py #753

Merged
merged 1 commit into from
Oct 31, 2023
Merged

Improved Documentation: Added comments in application/worker.py #753

merged 1 commit into from
Oct 31, 2023

Conversation

THEGAMECHANGER416
Copy link
Contributor

Added comments in difficult to understand areas in the file application/worker.py

  • What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
    This PR introduces code comments to improve code readability and understanding. It doesn't alter the functionality of the code but enhances its documentation.

  • Why was this change needed? (You can also link to an open issue here)
    This change was needed to make the code more comprehensible to developers, both current and future, who may need to work on or maintain this codebase. The added comments clarify the purpose of various code sections and components, making it easier to understand the logic and flow of the code.

  • Other information:
    No other significant changes are introduced in this PR, and it doesn't break any functionality. The goal is to enhance code documentation and maintainability.

Added comments in difficult to understand areas
Copy link

vercel bot commented Oct 31, 2023

@THEGAMECHANGER416 is attempting to deploy a commit to the Arc53 Team on Vercel.

A member of the Team first needs to authorize it.

Copy link

vercel bot commented Oct 31, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
docs-gpt ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 31, 2023 5:29pm
nextra-docsgpt ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 31, 2023 5:29pm

@THEGAMECHANGER416
Copy link
Contributor Author

@dartpain Thanks for suggesting this change. Done the update. Hope this gets approved now!
Thanks!

Copy link

codecov bot commented Oct 31, 2023

Codecov Report

Merging #753 (4c6b8b4) into main (fac8c9e) will not change coverage.
Report is 37 commits behind head on main.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main     #753   +/-   ##
=======================================
  Coverage   19.69%   19.69%           
=======================================
  Files          62       62           
  Lines        2844     2844           
=======================================
  Hits          560      560           
  Misses       2284     2284           
Files Coverage Δ
application/core/settings.py 100.00% <100.00%> (ø)
application/worker.py 29.68% <ø> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@THEGAMECHANGER416
Copy link
Contributor Author

@larinam can you also look into this PR?
Thanks

@dartpain dartpain merged commit e273da1 into arc53:main Oct 31, 2023
8 checks passed
@dartpain
Copy link
Contributor

Thank you @holopin-bot @THEGAMECHANGER416

Copy link

holopin-bot bot commented Oct 31, 2023

Congratulations @THEGAMECHANGER416, you just earned a badge! Here it is: https://holopin.io/claim/cloemrus960780fl11z4yyrex

This badge can only be claimed by you, so make sure that your GitHub account is linked to your Holopin account. You can manage those preferences here: https://holopin.io/account.
Or if you're new to Holopin, you can simply sign up with GitHub, which will do the trick!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
application Application
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants