Skip to content

v0.13.4#902

Closed
cching95 wants to merge 512 commits intomainfrom
develop
Closed

v0.13.4#902
cching95 wants to merge 512 commits intomainfrom
develop

Conversation

@cching95
Copy link
Copy Markdown
Collaborator

@cching95 cching95 commented Oct 6, 2025

No description provided.

dh1542 and others added 30 commits December 4, 2024 14:20
Signed-off-by: Dominik Hoffmann <dominik151099@outlook.de>
Signed-off-by: Dominik Hoffmann <dominik151099@outlook.de>
Signed-off-by: Dominik Hoffmann <dominik151099@outlook.de>
Signed-off-by: Dominik Hoffmann <dominik151099@outlook.de>
Signed-off-by: Dominik Hoffmann <dominik151099@outlook.de>
Signed-off-by: Dominik Hoffmann <dominik151099@outlook.de>
Signed-off-by: Dominik Hoffmann <dominik151099@outlook.de>
Signed-off-by: Dominik Hoffmann <dominik151099@outlook.de>
Signed-off-by: mollle <leonmariusmoll@gmail.com>
Signed-off-by: mollle <leonmariusmoll@gmail.com>
…entifyMissingDataIntervall and adjusted test_data.csv

Signed-off-by: mollle <leonmariusmoll@gmail.com>
Signed-off-by: Felipe Trost <felipe.trost@gmail.com>
Signed-off-by: Timm638 <Timm638@gmail.com>
Signed-off-by: Timm638 <Timm638@gmail.com>
Signed-off-by: Felipe Trost <felipe.trost@gmail.com>
This reverts commit 51c7178.

Signed-off-by: Felipe Trost <felipe.trost@gmail.com>
Signed-off-by: Timm638 <Timm638@gmail.com>
Signed-off-by: Timm638 <Timm638@gmail.com>
Signed-off-by: Dominik Hoffmann <dominik151099@outlook.de>
Signed-off-by: Dominik Hoffmann <dominik151099@outlook.de>
Signed-off-by: Dominik Hoffmann <dominik151099@outlook.de>
Signed-off-by: Dominik Hoffmann <dominik151099@outlook.de>
…Refactor-unit-tests

Feature/#62 interval filtering refactor unit tests
Signed-off-by: Timm638 <Timm638@gmail.com>
TugceOzberkYener and others added 23 commits March 18, 2025 09:52
* Package Updates

Signed-off-by: GBBBAS <bryce.bartmann@shell.com>

* Add check for langchain packages

Signed-off-by: GBBBAS <bryce.bartmann@shell.com>

* Fix linting error

Signed-off-by: GBBBAS <bryce.bartmann@shell.com>

---------

Signed-off-by: GBBBAS <bryce.bartmann@shell.com>
Signed-off-by: GBBBAS <42962356+GBBBAS@users.noreply.github.com>
* Package Updates

Signed-off-by: GBBBAS <bryce.bartmann@shell.com>

* Add check for langchain packages

Signed-off-by: GBBBAS <bryce.bartmann@shell.com>

---------

Signed-off-by: GBBBAS <bryce.bartmann@shell.com>
Signed-off-by: GBBBAS <42962356+GBBBAS@users.noreply.github.com>
* Update spelling of sort

Signed-off-by: GBBBAS <bryce.bartmann@shell.com>

* Update sonarcloud workflow

Signed-off-by: GBBBAS <bryce.bartmann@shell.com>

* Update micromamba environment

Signed-off-by: GBBBAS <bryce.bartmann@shell.com>

* Updates for Interpolate Query

Signed-off-by: GBBBAS <bryce.bartmann@shell.com>

* Updates to databricks-sdk

Signed-off-by: GBBBAS <bryce.bartmann@shell.com>

---------

Signed-off-by: GBBBAS <bryce.bartmann@shell.com>
* refactor summary query

Signed-off-by: cching95 <chloe.ching@shell.com>

* refactor summary query and unit test

Signed-off-by: cching95 <chloe.ching@shell.com>

* test fix

Signed-off-by: cching95 <chloe.ching@shell.com>

* undo test fix

Signed-off-by: cching95 <chloe.ching@shell.com>

* change CreateJob class to a dictionary and update code to get from dictionary

Signed-off-by: cching95 <chloe.ching@shell.com>

* update unit tests to remove CreateJobs class

Signed-off-by: cching95 <chloe.ching@shell.com>

* revert changes and add CreateJobs class

Signed-off-by: cching95 <chloe.ching@shell.com>

* update changes to example

Signed-off-by: cching95 <chloe.ching@shell.com>

* update databricks sdk version imports

Signed-off-by: cching95 <chloe.ching@shell.com>

* add sonar exceptions

Signed-off-by: cching95 <chloe.ching@shell.com>

* add protobuf version and sonar exceptions

Signed-off-by: cching95 <chloe.ching@shell.com>

* update sonar exclusions

Signed-off-by: cching95 <chloe.ching@shell.com>

* remove commented out code in summary query

Signed-off-by: cching95 <chloe.ching@shell.com>

---------

Signed-off-by: cching95 <chloe.ching@shell.com>
Signed-off-by: GBBBAS <42962356+GBBBAS@users.noreply.github.com>
Co-authored-by: GBBBAS <42962356+GBBBAS@users.noreply.github.com>
* refactored plot query

Signed-off-by: cching95 <chloe.ching@shell.com>

* Refactor Plot Query and Unit Tests

Signed-off-by: cching95 <chloe.ching@shell.com>

* Fix Plot unit test

Signed-off-by: cching95 <chloe.ching@shell.com>

---------

Signed-off-by: cching95 <chloe.ching@shell.com>
* fix time weighted average query to accept metadata as an option for step and remove default for window size mins in twa api

Signed-off-by: cching95 <chloe.ching@shell.com>

* fix mkdoc warnings

Signed-off-by: cching95 <chloe.ching@shell.com>

---------

Signed-off-by: cching95 <chloe.ching@shell.com>
* remove window_size_mins parameter from TWA

Signed-off-by: cching95 <chloe.ching@shell.com>

* add default value as metadata for step if not provided

Signed-off-by: cching95 <chloe.ching@shell.com>

---------

Signed-off-by: cching95 <chloe.ching@shell.com>
Comment on lines +69 to +79
needs: job_download_pr_artifact
uses: rtdip/core/.github/workflows/sonarcloud_reusable.yml@develop
with:
REPO_NAME: ${{ github.event.workflow_run.head_repository.full_name }}
HEAD_BRANCH: ${{ github.event.workflow_run.head_branch }}
HEAD_SHA: ${{ github.event.workflow_run.head_sha }}
PR_NUMBER: ${{ fromJSON(needs.job_download_pr_artifact.outputs.pr_info).pr_number }}
PR_HEAD_REF: ${{ fromJSON(needs.job_download_pr_artifact.outputs.pr_info).pr_head_ref }}
PR_BASE_REF: ${{ fromJSON(needs.job_download_pr_artifact.outputs.pr_info).pr_base_ref }}
secrets:
SONAR_TOKEN: ${{ secrets.SONAR_TOKEN }}

Check warning

Code scanning / CodeQL

Workflow does not contain permissions Medium

Actions job or workflow does not limit the permissions of the GITHUB_TOKEN. Consider setting an explicit permissions block, using the following as a minimal starting point: {}

Copilot Autofix

AI 10 months ago

To fix the issue, we need to add a permissions block to the workflow. This block should specify the minimal permissions required for the workflow to function correctly. Based on the workflow's operations (downloading artifacts, reading files, and running tests), the contents: read permission is sufficient. This ensures that the workflow can access repository contents without granting unnecessary write permissions.

The permissions block should be added at the root level of the workflow to apply to all jobs. Alternatively, it can be added to individual jobs if different jobs require different permissions.

Suggested changeset 1
.github/workflows/sonarcloud.yml

Autofix patch

Autofix patch
Run the following command in your local git repository to apply this patch
cat << 'EOF' | git apply
diff --git a/.github/workflows/sonarcloud.yml b/.github/workflows/sonarcloud.yml
--- a/.github/workflows/sonarcloud.yml
+++ b/.github/workflows/sonarcloud.yml
@@ -16,2 +16,5 @@
 
+permissions:
+  contents: read
+
 run-name: ${{ github.event.workflow_run.display_title }}
EOF
@@ -16,2 +16,5 @@

permissions:
contents: read

run-name: ${{ github.event.workflow_run.display_title }}
Copilot is powered by AI and may make mistakes. Always verify output.
@linux-foundation-easycla
Copy link
Copy Markdown

CLA Missing ID CLA Not Signed

@cching95 cching95 closed this Oct 6, 2025
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.