Skip to content

Added code and test for example component - #169

Open
naveena456 wants to merge 5 commits into
eclipse-score:mainfrom
bgsw-contrib:feature/add-example-component-code-tests
Open

Added code and test for example component#169
naveena456 wants to merge 5 commits into
eclipse-score:mainfrom
bgsw-contrib:feature/add-example-component-code-tests

Conversation

@naveena456

Copy link
Copy Markdown

Created an actual code and test target to demonstrate the usage of docs_bundle.

@github-actions

github-actions Bot commented Sep 3, 2026

Copy link
Copy Markdown
Contributor

License Check Results

🚀 The license check job ran with the Bazel command:

bazel run --lockfile_mode=error //:license-check

Status: ⚠️ Needs Review

Click to expand output
[License Check Output]
Extracting Bazel installation...
Starting local Bazel server (8.7.0) and connecting to it...
INFO: Invocation ID: 80f13d2c-b0c6-43d7-956f-f6ab2b8e3752
Computing main repo mapping: 
WARNING: For repository 'aspect_rules_lint', the root module requires module version aspect_rules_lint@1.10.2, but got aspect_rules_lint@2.3.0 in the resolved dependency graph. Please update the version in your MODULE.bazel or set --check_direct_dependencies=off
Loading: 
Loading: 0 packages loaded
Loading: 0 packages loaded
Loading: 0 packages loaded
    currently loading: 
Loading: 0 packages loaded
    currently loading: 
Analyzing: target //:license-check (1 packages loaded, 0 targets configured)
Analyzing: target //:license-check (1 packages loaded, 0 targets configured)

Analyzing: target //:license-check (37 packages loaded, 10 targets configured)

Analyzing: target //:license-check (88 packages loaded, 10 targets configured)

Analyzing: target //:license-check (99 packages loaded, 36 targets configured)

Analyzing: target //:license-check (162 packages loaded, 2841 targets configured)

Analyzing: target //:license-check (167 packages loaded, 5573 targets configured)

Analyzing: target //:license-check (167 packages loaded, 5573 targets configured)

Analyzing: target //:license-check (167 packages loaded, 5573 targets configured)

Analyzing: target //:license-check (172 packages loaded, 7574 targets configured)

Analyzing: target //:license-check (174 packages loaded, 10317 targets configured)

Analyzing: target //:license-check (174 packages loaded, 10317 targets configured)

Analyzing: target //:license-check (174 packages loaded, 10317 targets configured)

Analyzing: target //:license-check (175 packages loaded, 10437 targets configured)

INFO: Analyzed target //:license-check (176 packages loaded, 12016 targets configured).
[11 / 16] [Prepa] Expanding template external/score_tooling+/dash/tool/formatters/_dash_format_converter_stage2_bootstrap.py [for tool]
INFO: From Generating Dash formatted dependency file ...:
INFO: Successfully converted 2 packages from Cargo.lock to bazel-out/k8-fastbuild/bin/formatted.txt
[14 / 16] JavaToolchainCompileBootClasspath external/rules_java+/toolchains/platformclasspath.jar; 0s disk-cache, processwrapper-sandbox
[15 / 16] Building license.check.license_check.jar (); 0s disk-cache, multiplex-worker
INFO: Found 1 target...
Target //:license.check.license_check up-to-date:
  bazel-bin/license.check.license_check
  bazel-bin/license.check.license_check.jar
INFO: Elapsed time: 194.517s, Critical Path: 2.38s
INFO: 16 processes: 12 internal, 3 processwrapper-sandbox, 1 worker.
INFO: Build completed successfully, 16 total actions
INFO: Running command line: bazel-bin/license.check.license_check ./formatted.txt <args omitted>
usage: org.eclipse.dash.licenses.cli.Main [-batch <int>] [-cd <url>]
       [-confidence <int>] [-ef <url>] [-excludeSources <sources>] [-help] [-lic
       <url>] [-project <shortname>] [-repo <url>] [-review] [-summary <file>]
       [-timeout <seconds>] [-token <token>]

naveena456 and others added 4 commits September 3, 2026 14:43
Signed-off-by: Naveena Panne <naveenbellare@gmail.com>
Signed-off-by: Naveena Panne <pnn9kor@bosch.com>
Signed-off-by: Naveena Panne <pnn9kor@bosch.com>
@github-actions

github-actions Bot commented Sep 3, 2026

Copy link
Copy Markdown
Contributor

Documentation preview for this pull request is available at:
pr-169: https://eclipse-score.github.io/module_template/pr-169/

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟢 Approval recommended

The changes are small, consistent, and include a passing-focused unit test plus the intended Bazel wiring for documentation code targets.

Pull request overview

This PR adds a minimal C++ “component example” implementation plus a GoogleTest target, and wires the component into the docs_bundle so documentation can reference/link to real code targets.

Changes:

  • Added make_hello_message() implementation and public header under score/component_example/src/.
  • Added a cc_test validating the expected greeting output.
  • Updated score/component_example/BUILD to define a cc_library and connect it to the docs bundle via code_targets.
File summaries
File Description
score/component_example/tests/component_example_test.cpp Adds a GoogleTest verifying make_hello_message() output.
score/component_example/src/component_example.hpp Declares the example API function.
score/component_example/src/component_example.cpp Implements the example API function.
score/component_example/BUILD Introduces cc_library/cc_test targets and updates docs_bundle to use code_targets.
Review details
  • Files reviewed: 4/4 changed files
  • Comments generated: 0
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

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

Labels

None yet

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

2 participants