getsentry / getsentry/sentry-php

Exclude un-neccessary files from dist

Open Beginner friendly
#2,173 1 comment 0 reactions 0 assignees View on GitHub
Improvement PHP Waiting for: Product Owner
Dominant language
PHP
Stars
1.9k
Forks
474
Avg merge
2d 3h
Merged PRs (30d)
8

Description

### Problem Statement

Not needed runtime:
- `vendor/sentry/sentry/scripts/bump-version.sh`

May trigger vulnerability report, that package installed shell script, so better just skip it via .gitattributes

### Solution Brainstorm

Also review other files that don't deserve to be in composer dist.

Contributor guide

Open the contributing guide

Research direction

Start by checking the repository's .gitattributes rules and how the composer dist archive is assembled. Exclude vendor/sentry/sentry/scripts/bump-version.sh, then review the archive contents for other files that should not ship and verify the resulting dist package omits them.

Written by the indexing model from the issue text.

Assessment

Tech stack
php
Domain
build-system, release
Issue type
Refactor
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Quiet
Clarity
Mostly clear
Newbie friendliness
64/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.