Skip to content

Merge 'development' into 'main' #15

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 30 commits into
base: main
Choose a base branch
from
Open

Merge 'development' into 'main' #15

wants to merge 30 commits into from

Conversation

thegbk
Copy link
Member

@thegbk thegbk commented May 11, 2025

No description provided.

YazanShannak and others added 29 commits July 31, 2023 02:29

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
implement Nuha as a class using ONNX runtime

BREAKING CHANGE:
…s-to-the-prediction-results

feat: added original post and comments to the response
perf(requirements.txt): remove CUDA dependencies
refactor(src/model.py): remove unnecessary print
build(Dockerfile): Fix some issues in the Dockerfile
refactor(main.py-src/model.py): Refactor model output and response
add multi stage to the docker image

replace python image with python slim
* build: add GitHub Actions workflow to schedule milestones weekly

Signed-off-by: Tamim Hamoudi <[email protected]>

* move schedule-milestones.yaml from images to under workflows directly

Signed-off-by: Tamim Hamoudi <[email protected]>

---------

Signed-off-by: Tamim Hamoudi <[email protected]>
@github-advanced-security
Copy link

This pull request sets up GitHub code scanning for this repository. Once the scans have completed and the checks have passed, the analysis results for this pull request branch will appear on this overview. Once you merge this pull request, the 'Security' tab will show more code scanning analysis results (for example, for the default branch). Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results. For more information about GitHub code scanning, check out the documentation.

Comment on lines +9 to +24
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v2

- name: Schedule Milestones
uses: readmeio/[email protected]
id: scheduled
with:
token: ${{ secrets.GITHUB_TOKEN }}
title: 'S-'
days: Thursday
count: 4
format: YYYY-MM-DD

- name: Created Milestones
run: echo ${{ steps.scheduled.outputs.milestones }}

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: {contents: read}

Copilot Autofix

AI 22 days ago

To fix the issue, we need to add a permissions block to the workflow. Since the workflow uses the GITHUB_TOKEN to create milestones, it requires contents: read (to read repository contents) and issues: write (to create milestones). These permissions should be explicitly defined at the job level to ensure the workflow has only the necessary access.

The permissions block will be added under the generate job, specifying contents: read and issues: write.


Suggested changeset 1
.github/workflows/schedule-milestones.yaml

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/schedule-milestones.yaml b/.github/workflows/schedule-milestones.yaml
--- a/.github/workflows/schedule-milestones.yaml
+++ b/.github/workflows/schedule-milestones.yaml
@@ -8,2 +8,5 @@
   generate:
+    permissions:
+      contents: read
+      issues: write
     runs-on: ubuntu-latest
EOF
@@ -8,2 +8,5 @@
generate:
permissions:
contents: read
issues: write
runs-on: ubuntu-latest
Copilot is powered by AI and may make mistakes. Always verify output.
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.

5 participants