Skip to content

Conversation

lynxplay
Copy link
Contributor

@lynxplay lynxplay commented Jan 16, 2025

JSTs ignored prefixes are still pushed into the output file sink, which
is not needed when the input and output sources are the same.
The commit adds a stronger version of ignored prefixes, "hidden"
prefixes, which are not processed or pushed to the output sink.

This mainly aims to prevent potential write permission issues on
read-only files during in-place transformation.

Requires: #45

@neoforged-pr-publishing
Copy link

  • Publish PR to GitHub Packages

@Matyrobbrt Matyrobbrt added the enhancement New feature or request label Jan 16, 2025
@neoforged-automation neoforged-automation bot added the needs rebase This Pull Request needs to be rebased before being merged label Feb 16, 2025
@neoforged-automation
Copy link
Contributor

@lynxplay, this pull request has conflicts, please resolve them for this PR to move forward.

JSTs ignored prefixes are still pushed into the output file sink, which
is not needed when the input and output sources are the same.
The commit adds a stronger version of ignored prefixes, "hidden"
prefixes, which are not processed or pushed to the output sink.

This mainly aims to prevent potential write permission issues on
read-only files during in-place transformation.
@lynxplay lynxplay force-pushed the feature/hidden-prefixes branch from 542f5d5 to 8492caa Compare February 19, 2025 23:57
@neoforged-automation neoforged-automation bot removed the needs rebase This Pull Request needs to be rebased before being merged label Feb 19, 2025
@lynxplay
Copy link
Contributor Author

Rebased 🫡

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants