Skip to content

luasandbox: added two apis for luasandbox #947

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

luasandbox: added two apis for luasandbox #947

wants to merge 1 commit into from
+184 −1

Conversation

fr0m-scratch
Copy link
Contributor

This PR added two additional apis for the lua sandbox:

  1. gitGetBlobID to retrive blobID given refName and filePath.
  2. gitGetStagedFilePaths to retrive paths of all staged filesthat yet committed.

patzielinski
patzielinski reviewed May 5, 2025
View reviewed changes
@patzielinski
Copy link
Collaborator

@fr0m-scratch quick bump on this, did you still want to continue on this?

@fr0m-scratch fr0m-scratch mentioned this pull request Jul 9, 2025
Merged
@kusari-inspector Kusari Inspector
Copy link

kusari-inspector bot commented Jul 13, 2025 •
edited
Loading

Kusari Inspector

Kusari Analysis Results:

Proceed with these changes

✅ No Flagged Issues Detected
All values appear to be within acceptable risk parameters.

No pinned version dependency changes, code issues or exposed secrets detected!

Note

View full detailed analysis result for more information on the output and the checks that were run.

@kusari-inspector rerun - Trigger a re-analysis of this PR
@kusari-inspector feedback [your message] - Send feedback to our AI and team
See Kusari's documentation for setup and configuration.
Commit: cd7f93f, performed at: 2025-10-02T09:40:18Z

Found this helpful? Give it a 👍 or 👎 reaction!

@kusari-inspector Kusari Inspector
Copy link

Kusari PR Analysis rerun based on - 18ba318 performed at: 2025-07-13T11:08:29Z - link to updated analysis

patzielinski
patzielinski reviewed Sep 22, 2025
View reviewed changes
Copy link
Collaborator

@patzielinski patzielinski left a comment

Choose a reason for hiding this comment

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

Please open a PR with the two gitinterface changes (but make sure to keep this one still open). After we get that PR merged, please also rebase the PR to fix the failing CI, thanks!

@kusari-inspector Kusari Inspector
Copy link

Kusari PR Analysis rerun based on - ea00bf4 performed at: 2025-10-02T09:17:01Z - link to updated analysis

@kusari-inspector Kusari Inspector
Copy link

Kusari PR Analysis rerun based on - 5520f8f performed at: 2025-10-02T09:36:20Z - link to updated analysis

@kusari-inspector Kusari Inspector
Copy link

Kusari PR Analysis rerun based on - c36853a performed at: 2025-10-02T09:40:18Z - link to updated analysis

@kusari-inspector Kusari Inspector
Copy link

Kusari PR Analysis rerun based on - e9554bc performed at: 2025-10-02T09:42:51Z - link to updated analysis

@kusari-inspector Kusari Inspector
Copy link

Kusari PR Analysis rerun based on - ea00bf4 performed at: 2025-10-02T09:43:29Z - link to updated analysis

@kusari-inspector Kusari Inspector
Copy link

Kusari PR Analysis rerun based on - c2abf22 performed at: 2025-10-02T09:44:15Z - link to updated analysis

@fr0m-scratch
sandbox: added two apis for luasandbox
cd7f93f 
Added `gitGetBlobID` to retrive blobID given refName and filePath
Added `gitGetStagedFilePaths` to retrive paths of all staged filesthat yet committed.

Signed-off-by: Jamal Cao <jamaltheodds@gmail.com>
@kusari-inspector Kusari Inspector
Copy link

Kusari PR Analysis rerun based on - cd7f93f performed at: 2025-10-02T10:01:07Z - link to updated analysis

@patzielinski patzielinski self-requested a review October 2, 2025 13:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@patzielinski patzielinski Awaiting requested review from patzielinski

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@fr0m-scratch @patzielinski