envoyproxy / envoyproxy/envoy

PLACEHOLDER still problematic on main (at least) for branch dev/reopen

Open
#46,149 3 comments 0 reactions 0 assignees View on GitHub
area/release bug no stalebot
Dominant language
C++
Stars
28.9k
Forks
5.6k
Avg merge
1d 20h
Merged PRs (30d)
437

Description

a fix was added in toolshed to add a placeholder when reopening branches

this added or ommitted .txt and didnt work - i thought i had rolled out a fix - but perhaps only oin release branches

the better fix is to remvoe the legacy changelog code and the need for the placeholder at all

Contributor guide

Open the contributing guide

Research direction

Start by tracing the toolshed logic used when reopening branches, especially the legacy changelog handling and placeholder creation for dev/reopen on main. Remove the legacy changelog code and placeholder requirement, then verify that reopening the branch no longer adds or omits a .txt placeholder.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp
Domain
release, tooling
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Quiet
Clarity
Needs clarification
Newbie friendliness
42/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.