Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove extra modelines from included source files #36

Open
jayvdb opened this issue Jun 5, 2018 · 3 comments
Open

Remove extra modelines from included source files #36

jayvdb opened this issue Jun 5, 2018 · 3 comments
Milestone

Comments

@jayvdb
Copy link
Member

jayvdb commented Jun 5, 2018

The modeline on the output file should appear only once at the top of the file, and any subsequent modelines should be removed.

https://www.python.org/dev/peps/pep-0263/
https://wiki.python.org/moin/Vim

When templates include another template which has a modeline, multiple modelines may appear.

@chfw
Copy link
Member

chfw commented Jun 5, 2018

similar to your previous request to remove trailing newlines.

@chfw
Copy link
Member

chfw commented Jul 15, 2018

removing multiple mode lines should be a custom operation.

@jayvdb
Copy link
Member Author

jayvdb commented Jan 24, 2019

Similar to #38 , a plugin.

@jayvdb jayvdb added this to the 0.5.0 milestone Jan 24, 2019
chfw added a commit that referenced this issue May 24, 2020
* ✨enabled plugins dir via command line

* 📚 update exit code

* 🔬 test -pd on command line

* 📚 update readme

* 🐛 fix unit test which failed to detect the missing feature

* 🔨 code refactor copy engine and strip content processor

* 📚 here is the way to write modelines remover. #36

* 🔨 code refactoring
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants