Skip to content

precedence for isReadonly() for #181708#181955

Merged
bpasero merged 2 commits intomicrosoft:ben/readonly-filesfrom
jackpunt:ben-readonly-files4
May 10, 2023
Merged

precedence for isReadonly() for #181708#181955
bpasero merged 2 commits intomicrosoft:ben/readonly-filesfrom
jackpunt:ben-readonly-files4

Conversation

@jackpunt
Copy link
Contributor

@jackpunt jackpunt commented May 9, 2023

#181708

isReadonly precedence:

  • inSession/Command
  • readonlyInclude/Exclude Globs
  • configuredReadonlyFromPermissions && stat?.locked
  • false

see: #161716 (comment)

@bpasero bpasero added this to the May 2023 milestone May 10, 2023
@bpasero bpasero merged commit 05632b6 into microsoft:ben/readonly-files May 10, 2023
@github-actions github-actions bot locked and limited conversation to collaborators Jun 24, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants