Skip to content

Conversation

@mraible
Copy link
Contributor

@mraible mraible commented Jan 15, 2026

Replace the dorny/paths-filter action with native git commands for detecting path changes. This action is not ProdSec approved.

The replacement script:

  • Handles both pull_request and push events
  • Compares the appropriate SHAs based on event type
  • Handles edge case of initial pushes (no previous commit)
  • Produces identical outputs (functions and ui booleans)

Use native git commands to detect path changes instead of the
dorny/paths-filter action, which is not ProdSec approved.

The replacement script handles both pull_request and push events,
and includes handling for initial pushes where there is no previous
commit to compare against.
@mraible mraible requested a review from a team January 15, 2026 15:35
prvn
prvn previously approved these changes Jan 15, 2026
@mraible mraible force-pushed the replace-paths-filter branch 3 times, most recently from b0bc704 to 54df76f Compare January 15, 2026 16:42
Move the integration tests that require Falcon API credentials from
main.yml to a standalone test-integration.sh script. This allows:

- CI to run without secrets (unit tests only)
- Developers to run integration tests locally with their own credentials
- The script prompts for credentials if not set as environment variables

The blog post can now reference the standalone script instead of the
workflow file for demonstrating integration test automation.
@mraible mraible force-pushed the replace-paths-filter branch from 54df76f to 4b4e6ff Compare January 15, 2026 16:43
@mraible
Copy link
Contributor Author

mraible commented Jan 15, 2026

This should not be merged until the blog post is updated to point to the integration script.

- Auto-create and activate virtual environment
- Install dependencies automatically
- Remove FALCON_BASE_URL (FalconPy defaults to us-1)
- Document prerequisites: deployed app and API scopes
@mraible mraible requested a review from a team January 15, 2026 17:49
@mraible mraible merged commit a6695d4 into main Jan 26, 2026
14 checks passed
@mraible mraible deleted the replace-paths-filter branch January 26, 2026 18:48
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