Heresy refactor #2
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…(jj2cli). The upstream [1] appears to have fallen in disrepair. In the meantime, this fork has implemented additional functionality and alternative (and arguably more flexible) approaches to some use cases. At this point, a merge with the upstream is unlikely to happen and it will become even more likely with each future commit. Starting with this commit, this fork will be developed independently and under a new name to reflect the additional capabilities (and incompatibilities). [1] https://github.com/kolypto/j2cli
Removed the obsolete implementation of read_context_data().
Convert return data to dict. Down the line, the data are iterated using six.iteritems(), which does not support iterating over a filter.
Allows selecting any of the available Jinja2 behaviours (strict, normal, debug) for handling missing variables.
Useful for aligning trailing line comments in a block of text.
Allows looking up a string in the context.
…c behaviour!
The old code was making an attempt to work with any part of the
data spec missing:
- data_in:json would interpret the contents of data_in as json.
- data_in:foo would interpret the contents of data_in as the
fallback format, and attach them to "foo" in the context.
This resulted in the code being complicated, and also not working
as expected in some cases. For this, it was decided to simplify the
data spec format as <source>|[<source>]:[<format>[:<context_dest]].
Any parts not-required can be ommitted.
The previous examples in the new format can be written as:
- data_in:json
- data_in::foo or data_in:?:foo
The order of format and context_dest parts has been reversed, as the
format part will probably be used more often.
m000
force-pushed
the
heresy-refactor
branch
6 times, most recently
from
16a4bec
to
2f711ae
Compare
Discontinued: - setuptools (using poetry and pyproject.toml from now on) - Travis CI (won't bother contacting support to fix my account [1]) - nose (does not seem to be actively maintained [2]) - six ( - .python-version (used by pyenv) Updated: - tox configuration to use Python >=3.10 - .gitignore - make targets for development/testing Now using: - poetry - github actions - pytest [1] https://archive.ph/lOFGT [2] https://pypi.org/project/nose/#history
…README.md. Extended documentation moved under docs directory.
m000
force-pushed
the
heresy-refactor
branch
6 times, most recently
from
6229fde
to
3e07fd7
Compare
m000
force-pushed
the
heresy-refactor
branch
from
335381a
to
982f620
Compare
m000
force-pushed
the
heresy-refactor
branch
from
10460f5
to
0bf0926
Compare
m000
force-pushed
the
heresy-refactor
branch
from
a5bbbc4
to
4392794
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.