Skip to content

basic: add autoMakeExecutable - #1858

Closed
gs-101 wants to merge 1 commit into
NotAShelf:mainfrom
gs-101:autoMakeExecutable
Closed

gs-101 wants to merge 1 commit into
NotAShelf:mainfrom
gs-101:autoMakeExecutable

Conversation

@gs-101

@gs-101 gs-101 commented Nov 1, 2026

Copy link
Copy Markdown

Add an option to automatically mark files with a shebang as executable on save.

Sanity Checking

  • I have updated the changelog as per my changes
  • I have tested, and self-reviewed my code
  • My changes fit guidelines found in hacking nvf
  • Style and consistency
    • I ran Alejandra to format my code (nix fmt)
    • My code conforms to the editorconfig configuration of the project
    • My changes are consistent with the rest of the codebase
  • If new changes are particularly complex:
    • My code includes comments in particularly complex areas
    • I have added a section in the manual
    • (For breaking changes) I have included a migration guide
  • Package(s) built:
    • .#nix (default package)
    • .#maximal
    • .#docs-html (manual, must build)
    • .#docs-linkcheck (optional, please build if adding links)
  • Tested on platform(s)
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin

Add a 👍 reaction to pull requests you find important.

Add an option to automatically mark files with a shebang as executable
on save.
github-actions Bot pushed a commit that referenced this pull request Nov 1, 2026
@github-actions

github-actions Bot commented Nov 1, 2026

Copy link
Copy Markdown

🚀 Live preview deployed from 0ae8788

View it here:

Debug Information

Triggered by: gs-101

HEAD at: autoMakeExecutable

Reruns: 4213

@horriblename

Copy link
Copy Markdown
Collaborator

I don't think this fits in NVF, as it is too niche a feature. From a user perspective, if I wanted such a feature, writing the autocmd myself is probably easier than trying to look it up in the docs

@gs-101
gs-101 deleted the autoMakeExecutable branch November 1, 2026 16:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants