Skip to content

adding filter - #180

Closed
Saumya-R wants to merge 1 commit into
eclipse-score:mainfrom
qorix-group:saumya_codeql_exclusion
Closed

Saumya-R wants to merge 1 commit into
eclipse-score:mainfrom
qorix-group:saumya_codeql_exclusion

Conversation

@Saumya-R

@Saumya-R Saumya-R commented Mar 11, 2026 •

Copy link
Copy Markdown
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.

@github-actions

Copy link
Copy Markdown

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

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.

Pull request overview

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.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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.

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 3 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@PiotrKorkus PiotrKorkus 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.

report is empty

@Saumya-R
Saumya-R marked this pull request as draft March 12, 2026 07:48
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
@Saumya-R
Saumya-R force-pushed the saumya_codeql_exclusion branch from 2583aa4 to 0ba3b57 Compare March 12, 2026 09:53
@Saumya-R

Copy link
Copy Markdown
Contributor Author

Dropping the PR , will create a new one.

@Saumya-R Saumya-R closed this Mar 12, 2026
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.

3 participants