Skip to content

Remove usage of pull_request_target everywhere#878

Merged
samikshya-db merged 2 commits into
databricks:mainfrom
samikshya-db:samikshya-chand_data/security2
Jul 2, 2025
Merged

Remove usage of pull_request_target everywhere#878
samikshya-db merged 2 commits into
databricks:mainfrom
samikshya-db:samikshya-chand_data/security2

Conversation

@samikshya-db
Copy link
Copy Markdown
Collaborator

@samikshya-db samikshya-db commented Jul 2, 2025

Description

  • Given that GITHUB TOKEN is also unsafe to use in forked PRs (even though it is rotated automatically), I will remove these too
  • Modified the tests accordingly.

Testing

Additional Notes to the Reviewer

NO_CHANGELOG=true

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jul 2, 2025

📊 Code Coverage Report

Overall Project 86.18% 🟢

There is no coverage information present for the Files changed

@samikshya-db samikshya-db requested a review from jprakash-db July 2, 2025 11:02
Copy link
Copy Markdown
Collaborator

@jprakash-db jprakash-db left a comment

Choose a reason for hiding this comment

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

LGTM

@samikshya-db samikshya-db merged commit 9bf1d27 into databricks:main Jul 2, 2025
12 of 15 checks passed
@samikshya-db samikshya-db deleted the samikshya-chand_data/security2 branch July 2, 2025 11:02
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