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

[Enhancement] Add Support for Common 2nd Suffixes (Fix #84) (#6) #85

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

kevinmatthes
Copy link
Contributor

When working on large projects, one often utilises several automation techniques in order to abbreviate redundant tasks, such as the build instruction specification by hand over and over again for every single build process. Another common example is testing where not only unit and integration but also mutation tests can be specified.

In order to set up such an automation, one needs to apply the conventions of the respective build system. The autotools suite, for instance, expects some input files to have the *.in suffix and will generate processed source files with the *.out suffix. On the other hand, some build steps may be destructive such that the system itself needs to take care to prevent a loss of progress. mutmut, a Python library for mutation testing, for example, creates backups of the respective source files which have the *.bak suffix.

All those systems have in common that one might want to take a look into the newly generated files for getting an idea which changes the respective system has applied. But then, Zee currently falls back to the plain text mode without any highlighting since the suffix could not be matched if not explicitly specified in the configuration.

This Pull Request challenges this disadvantage by the introduction of a matching against the common second suffixes named above by default against each file. Furthermore, already configured second suffixes were removed from the configuration since they are implicit by this change, now.

…#6)

* [Enhancement] Add Support for Common 2nd Suffixes

* Remove obsolete suffixes from Python entry
@kevinmatthes
Copy link
Contributor Author

I admit that this is not the most performant solution but as a first suggestion, it replaces the previous obligation to add any suffix combination by hand. I am working on a more efficient solution which I am going to submit when I finished. Until then, this workaround seems to be not the worst option, at least in my opinion, such that I consider it ready for merging, anyway.

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

Successfully merging this pull request may close these issues.

1 participant