Skip to content

RDKCOM-5541: RDKBDEV-3388:Add rdk logger Documentation#56

Closed
Ashrafbhanu-k wants to merge 3 commits into
rdkcentral:developfrom
Ashrafbhanu-k:feature/RDKBDEV-3388
Closed

RDKCOM-5541: RDKBDEV-3388:Add rdk logger Documentation#56
Ashrafbhanu-k wants to merge 3 commits into
rdkcentral:developfrom
Ashrafbhanu-k:feature/RDKBDEV-3388

Conversation

@Ashrafbhanu-k

Copy link
Copy Markdown

RDKBDEV-3388

Reason for Change:
To add Component Documentation for rdk logger
Fix:
Added the documentation
Test Procedure:
None
Signed-off-by: Ashrafbhanu-k [Ashraf_bhanu@comcast.com]

@Ashrafbhanu-k
Ashrafbhanu-k requested a review from a team as a code owner March 3, 2026 05:58
@github-actions

github-actions Bot commented Mar 3, 2026

Copy link
Copy Markdown

All contributors have signed the CLA ✍️ ✅
Posted by the CLA Assistant Lite bot.

@Ashrafbhanu23

Copy link
Copy Markdown

recheck

@Ashrafbhanu23

Copy link
Copy Markdown

/recheck

@Ashrafbhanu23

Copy link
Copy Markdown

I have read the CLA Document and I hereby sign the CLA

@AkhilaReddyK7

Copy link
Copy Markdown

@karuna2git
Can you please review and merge the PR.

@Ashrafbhanu-k

Copy link
Copy Markdown
Author

Hi @karuna2git
i have addressed review comments and updated README under /docs, could you please check and merge this updated PR?

@AkhilaReddyK7 AkhilaReddyK7 added the community-contribution Contribution from community label Mar 17, 2026
@AkhilaReddyK7 AkhilaReddyK7 changed the title RDKBDEV-3388:Add rdk logger Documentation RDKCOM-5541: RDKBDEV-3388:Add rdk logger Documentation Mar 17, 2026
@AkhilaReddyK7 AkhilaReddyK7 added the documentation Improvements or additions to documentation label Apr 2, 2026
@tinaelizabeth84

Copy link
Copy Markdown

@Ashrafbhanu-k
Comment from component owner

  1. I see that the documentation is very specific to RDKB and how to use it. This will be changed in 2604.
  2. The existing documents are deleted but pl restore it.
  3. The new document that must be talking about how rdklogger works; how log4c is fitting within rdklogger; and how it is portable to replace underneath log4c with any new library.
  4. Remember, RDKB is one of the profile that uses it. But we do have RDK-V, RDK-E, RDK-C using the same.

Thanks,
Karunakaran A

@Ashrafbhanu-k

Ashrafbhanu-k commented May 4, 2026

Copy link
Copy Markdown
Author

@Ashrafbhanu-k Comment from component owner

  1. I see that the documentation is very specific to RDKB and how to use it. This will be changed in 2604.
  2. The existing documents are deleted but pl restore it.
  3. The new document that must be talking about how rdklogger works; how log4c is fitting within rdklogger; and how it is portable to replace underneath log4c with any new library.
  4. Remember, RDKB is one of the profile that uses it. But we do have RDK-V, RDK-E, RDK-C using the same.

Thanks, Karunakaran A

Hi @tinaelizabeth84
Latest updated amended PR 90f805b here we added updated README after fixing review comments. The old files are kept untouched. Based on an internal discussion with CO, we are holding this PR for now.

@pradeeptakdas

Copy link
Copy Markdown

Hi @Ashrafbhanu-k

Please let us know once you fix the review comments.

@gourivarma3

Copy link
Copy Markdown

Created a generic documentation for rdklogger (both RDKB and RDKV) as requested. Please find the new pr link and the corresponding RDKDEV ticket below

#58
RDKDEV-1442

Therefore, Closing this PR.

@github-actions github-actions Bot locked and limited conversation to collaborators Jul 15, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

community-contribution Contribution from community documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants