Skip to content

codeql: removing the coding-standard repo#183

Open
Saumya-R wants to merge 1 commit intoeclipse-score:mainfrom
qorix-group:saumya_codeql_exclusion
Open

codeql: removing the coding-standard repo#183
Saumya-R wants to merge 1 commit intoeclipse-score:mainfrom
qorix-group:saumya_codeql_exclusion

Conversation

@Saumya-R
Copy link
Contributor

This PR adds a post-processing filter step to the SARIF recategorization workflow so that only results associated with paths under repos/ are kept in the final SARIF output.

Changes:

Runs jq after recategorization to filter SARIF runs[].results[] by artifactLocation.uri.
Overwrites the original SARIF with the filtered output.

@Saumya-R Saumya-R force-pushed the saumya_codeql_exclusion branch from 0ba3b57 to 17bff5d Compare March 12, 2026 10:05
@github-actions
Copy link

The created documentation from the pull request is available at: docu-html

Copy link
Contributor

@PiotrKorkus PiotrKorkus left a comment

Choose a reason for hiding this comment

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

the number of warnings reduced from 84 to 1. This is incorrect.
We just want to remove codeql-coding-standards-repo/* from a links to particular issues

@Saumya-R Saumya-R marked this pull request as draft March 12, 2026 19:23
@Saumya-R
Copy link
Contributor Author

Branching from the main eclipse reference_integration repo and generating the CodeQL multi repo scan , only one warning is being shown .
Reference run : https://github.com/eclipse-score/reference_integration/actions/runs/23020119551

adding conditional jq

chore: extend codeowners list (eclipse-score#175)

* chore: extend codeowners list

* fix: no commas

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Signed-off-by: Alexander Lanin <alex@lanin.de>

---------

Signed-off-by: Alexander Lanin <alex@lanin.de>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Integrates datarouter  (eclipse-score#178)

* Integrates datarouter

- Deploys datarouetr in linux x86_64 and
qnx_x86_64 image
- Multicast route added to qnx guest in
network_setup_dhcp.sh required for capturing
DLTs.
- dlt_config added for x86_64 and qnx x86_64
targets that includes the vlan_address updated
for dlt_receive to capture untagged ethernet frames
on tap0 interface

* Adds datarouter test_remote_logging integration test

- Adds test_remote_logging integration test that
verifies remote logging of datarouter and
network config. of the linux x86_64 and
qnx qemu image by capturing DLT logs on the host.

Use py_itf_test rule with select support (eclipse-score#177)

- Update score_itf to 44c75de and score_bazel_platforms to b72a5c3 in
  known_good.json and MODULE.bazel overrides
- Add explicit bazel_dep on platforms 1.0.0 (required by new config_settings)
- Merge the two py_itf_test targets (linux_x86_64, qnx_x86_64) in
  feature_integration_tests/itf/BUILD into a single 'itf' target that uses
  config_setting + select() to switch args/data/plugins per platform;
  keep backward-compatible aliases for the old target names
- Upgrade rules_oci from 1.8.0 to 2.2.7; adapt docker images
  to use oci_load (replacing the removed oci_tarball) and add the
  platform-specific repo variants

revert the filters to test

adding jd check and filter

regex matches relative path

removing the filter to test

codeql: removing the coding-standard repo

adding conditional jq

chore: extend codeowners list (eclipse-score#175)

* chore: extend codeowners list

* fix: no commas

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Signed-off-by: Alexander Lanin <alex@lanin.de>

---------

Signed-off-by: Alexander Lanin <alex@lanin.de>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Integrates datarouter  (eclipse-score#178)

* Integrates datarouter

- Deploys datarouetr in linux x86_64 and
qnx_x86_64 image
- Multicast route added to qnx guest in
network_setup_dhcp.sh required for capturing
DLTs.
- dlt_config added for x86_64 and qnx x86_64
targets that includes the vlan_address updated
for dlt_receive to capture untagged ethernet frames
on tap0 interface

* Adds datarouter test_remote_logging integration test

- Adds test_remote_logging integration test that
verifies remote logging of datarouter and
network config. of the linux x86_64 and
qnx qemu image by capturing DLT logs on the host.

Use py_itf_test rule with select support (eclipse-score#177)

- Update score_itf to 44c75de and score_bazel_platforms to b72a5c3 in
  known_good.json and MODULE.bazel overrides
- Add explicit bazel_dep on platforms 1.0.0 (required by new config_settings)
- Merge the two py_itf_test targets (linux_x86_64, qnx_x86_64) in
  feature_integration_tests/itf/BUILD into a single 'itf' target that uses
  config_setting + select() to switch args/data/plugins per platform;
  keep backward-compatible aliases for the old target names
- Upgrade rules_oci from 1.8.0 to 2.2.7; adapt docker images
  to use oci_load (replacing the removed oci_tarball) and add the
  platform-specific repo variants

revert the filters to test

adding jd check and filter

regex matches relative path

removing the filter to test

adding the filter
@Saumya-R Saumya-R force-pushed the saumya_codeql_exclusion branch from d54ce2c to e9c86f7 Compare March 13, 2026 05:17
@Saumya-R Saumya-R marked this pull request as ready for review March 13, 2026 05:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants