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.

Sign up for GitHub

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

Jump to bottom

Improvement in code-comments and readability #61

Closed
TrekkieCoder opened this issue Aug 20, 2022 · 1 comment
Closed

Improvement in code-comments and readability #61

TrekkieCoder opened this issue Aug 20, 2022 · 1 comment
Assignees
@PacketCrunch
Labels
documentation Improvements or additions to documentation

Comments

@TrekkieCoder
Copy link
Collaborator

I have noticed that some code-comments are either non intuitive or not present at all .It would be great if this can be addressed !!
PacketCrunch added a commit that referenced this issue Aug 20, 2022
@PacketCrunch
gh-61 improved code comments patch1
ff329d9
PacketCrunch added a commit that referenced this issue Aug 21, 2022
@PacketCrunch
gh-61 improved code comments and logs patch2
09c6834
PacketCrunch added a commit that referenced this issue Aug 23, 2022
@PacketCrunch
gh-61 reorganized files for better readability and structure
e64a33c
PacketCrunch added a commit that referenced this issue Aug 23, 2022
@PacketCrunch
gh-61 renamed var names for better readability and structure
2551fc0
PacketCrunch added a commit that referenced this issue Aug 23, 2022
@PacketCrunch
Merge branch 'main' into gh-61
a8ffacc
PacketCrunch added a commit that referenced this issue Aug 23, 2022
@PacketCrunch
Merge pull request #62 from loxilb-io/gh-61
2f867d8 
Gh-61 Pull request
PacketCrunch added a commit that referenced this issue Sep 2, 2022
@PacketCrunch
gh-61 Added some code comments
2fef032
PacketCrunch added a commit that referenced this issue Sep 2, 2022
@PacketCrunch
gh-61 gofmt applied
4fa4cea
PacketCrunch added a commit that referenced this issue Sep 2, 2022
@PacketCrunch
gh-61 gofmt applied - Fixed unused import
0e5f494
PacketCrunch added a commit that referenced this issue Sep 3, 2022
@PacketCrunch
gh-61 Continuing to add comments where necessary and helpful
34232a5
@PacketCrunch PacketCrunch removed this from the First Beta Release 0.7.0 milestone Sep 3, 2022
PacketCrunch added a commit that referenced this issue Sep 5, 2022
@PacketCrunch
gh-61 Code indentation
c5b218c
PacketCrunch added a commit that referenced this issue Sep 6, 2022
@PacketCrunch
gh-61 changes as per golint
25b8626
PacketCrunch added a commit that referenced this issue Sep 6, 2022
@PacketCrunch
gh-61 changes as per golint -p2
1ef1695
PacketCrunch added a commit that referenced this issue Sep 6, 2022
@PacketCrunch
gh-61 changes as per golint -p3
db734b7
PacketCrunch added a commit that referenced this issue Sep 14, 2022
@PacketCrunch
gh-61 changes as per golint -p4
a9414ee
PacketCrunch added a commit that referenced this issue Sep 14, 2022
@PacketCrunch
gh-61 golint fixups -p5
1d1cb5b
PacketCrunch added a commit that referenced this issue Sep 15, 2022
@PacketCrunch
gh-61 golint fixups -p6
7b88ae7
PacketCrunch added a commit to PacketCrunch/loxilb-next that referenced this issue Nov 9, 2022
@PacketCrunch
loxilb-iogh-61 Changes as per gofmt
e74f68f
PacketCrunch added a commit to PacketCrunch/loxilb-next that referenced this issue Nov 28, 2022
@PacketCrunch
loxilb-iogh-61 fixes as per golint
9cbe0ed
PacketCrunch added a commit to PacketCrunch/loxilb-next that referenced this issue Dec 1, 2022
@PacketCrunch
loxilb-iogh-61 gofmt changes
265479f
PacketCrunch added a commit to PacketCrunch/loxilb-next that referenced this issue Dec 13, 2022
@PacketCrunch
loxilb-iogh-61 some gmfmt changes
01e8785
PacketCrunch added a commit to PacketCrunch/loxilb-next that referenced this issue Dec 22, 2022
@PacketCrunch
loxilb-iogh-61 gofmt suggested changes
1868410
@UltraInstinct14 UltraInstinct14 added documentation Improvements or additions to documentation and removed status:always open labels Dec 30, 2022
@UltraInstinct14
Copy link
Contributor

All processes are now in place to ensure this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@PacketCrunch PacketCrunch

Labels
documentation Improvements or additions to documentation
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@UltraInstinct14 @PacketCrunch @TrekkieCoder