Skip to content

stkh_req and feat_req as code not as need - #162

Draft
anmittag wants to merge 2 commits into
eclipse-score:mainfrom
etas-contrib:remove_stkhreq
Draft

stkh_req and feat_req as code not as need#162
anmittag wants to merge 2 commits into
eclipse-score:mainfrom
etas-contrib:remove_stkhreq

Conversation

@anmittag

@anmittag anmittag commented Aug 27, 2026

Copy link
Copy Markdown
Member

Result

a) there is none of the req unintenionally in the Cross Repo Metrics Report
b) Modules using the template are avoiding to create one on module level unintentionally

Comment thread examples/docs/architecture_modeling_example.rst Outdated
Comment thread examples/docs/architecture_modeling_example.rst Outdated
@github-actions

github-actions Bot commented Aug 27, 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: 8dda1775-6360-4e09-92f7-2a94a62c61db
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)
Analyzing: target //:license-check (1 packages loaded, 0 targets configured)
Analyzing: target //:license-check (1 packages loaded, 0 targets configured)

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

Analyzing: target //:license-check (92 packages loaded, 23 targets configured)

Analyzing: target //:license-check (162 packages loaded, 4895 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 (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)

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

INFO: Analyzed target //:license-check (176 packages loaded, 12016 targets configured).
[12 / 16] checking cached actions
INFO: From Generating Dash formatted dependency file ...:
INFO: Successfully converted 2 packages from Cargo.lock to bazel-out/k8-fastbuild/bin/formatted.txt
[15 / 16] [Prepa] Building license.check.license_check.jar ()
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: 188.531s, Critical Path: 2.15s
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>]

@anmittag
anmittag marked this pull request as draft August 27, 2026 14:35
@anmittag

Copy link
Copy Markdown
Member Author

needs more to be changed then removing the two need :(

@MaximilianSoerenPollak

Copy link
Copy Markdown
Contributor

needs more to be changed then removing the two need :(

As the old needs were referenced in some other needs & places you either need to remove those references, or what you can also do is duplicate the thing.

Have it one time as a code-block and once as a real need and if you don't want it rendered theoretically you could add `:hidden: as an option (if that is still available in the newest version).

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.

3 participants