Skip to content

feat: add separate permission to check for writable files#1455

Merged
susnux merged 1 commit intomainfrom
feat/writable
Jan 12, 2026
Merged

feat: add separate permission to check for writable files#1455
susnux merged 1 commit intomainfrom
feat/writable

Conversation

@susnux
Copy link
Contributor

@susnux susnux commented Jan 12, 2026

There is a difference if a file is only updateable (allow to update it means rename or move) or if the file is writable (content can be changed).
Needed to use erasable syntax as Typescript enums do not allow entries with the same value.

@susnux susnux requested review from CarlSchwan and artonge January 12, 2026 14:50
@susnux susnux added type: enhancement 🚀 New feature or request 3. to review 3️⃣ Waiting for reviews labels Jan 12, 2026
@susnux susnux marked this pull request as ready for review January 12, 2026 14:50
There is a difference if a file is only updatable (allow to update it
means rename or move) or if the file is writable (content can be
changed).

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
@codecov
Copy link

codecov bot commented Jan 12, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 86.42%. Comparing base (9febf8f) to head (62d4dd8).
⚠️ Report is 10 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1455      +/-   ##
==========================================
- Coverage   86.72%   86.42%   -0.31%     
==========================================
  Files          27       27              
  Lines         723      707      -16     
  Branches      196      197       +1     
==========================================
- Hits          627      611      -16     
  Misses         82       82              
  Partials       14       14              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@susnux susnux merged commit 00c28f6 into main Jan 12, 2026
12 checks passed
@susnux susnux deleted the feat/writable branch January 12, 2026 15:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review 3️⃣ Waiting for reviews type: enhancement 🚀 New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants