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

better formatting of generator and generated file in orer to increase… #1425

Merged
merged 1 commit into from Feb 29, 2016

Conversation

jensens
Copy link
Sponsor Member

@jensens jensens commented Feb 29, 2016

… readability of Python code and generated Grundfile.js

modifyVars: {{
{globalVars}
{globalVars}
}}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why this unindent?

Copy link
Sponsor Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

because I indent manually and then it fits afterwards (before only first line was indented).

@thet
Copy link
Member

thet commented Feb 29, 2016

I guess, that's untested code, isn't it?
I'll let run a Pull-Request job anyways: http://jenkins.plone.org/job/pull-request-5.0/988/

@jensens
Copy link
Sponsor Member Author

jensens commented Feb 29, 2016

I dont see any test for it, anyway its also difficult to test. This would need quite a lot of refactoring and out of scope of this PR.

thet added a commit that referenced this pull request Feb 29, 2016
better formatting of generator and generated file in orer to increase…
@thet thet merged commit c3e0a5d into master Feb 29, 2016
@thet thet deleted the jensens-gruntgen-cleanup branch February 29, 2016 13:01
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.

None yet

3 participants