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

template cleanup for pike #44

Open
stdweird opened this issue Aug 10, 2017 · 4 comments
Open

template cleanup for pike #44

stdweird opened this issue Aug 10, 2017 · 4 comments

Comments

@stdweird
Copy link
Member

there obviously has been a huge effort in creating the current templates, but it looks like there is the possibility for some cleanup.
we are willing to provide some effort here but we have some questions:

  • we don't have any setup now, so we can start with an ocata branch. but the cleanup i envsion is non-trivial, and existing sites might want ot switch to ocata before; so i propose we create a pike branch and start there. we will try to follow up the work for ocata and adjust were needed

wrt cleanup, there are some things i would like to see changed:

  • some syntax / style cleanup

    • i guess this is not that controversial, we could do that in an ocata branch
  • make a component to address a few issues:

  • there are scripts (long, non-trivial ones, no set -e, lots of copy/paste)

  • TT rendering: metaconfig with tiny is nice, but some additional structure in the schema seems a good thing, which requires non-trivial rendering

  • there's a lot of extensible in the schema and types. i'm aware that having the full schema might too much, but schema vaiidation is a good thing to have. maybe we can generate the schema from the source or documentation?

@apdibbo
Copy link
Contributor

apdibbo commented Aug 10, 2017

Most of this is stuff that I would like to do anyway but have limited bandwidth to do the work.

I am hoping to be moving to Ocata soon so would be happy to help with the work and testing.

Additional structure and schema work would be good

@stdweird
Copy link
Member Author

@apdibbo can someone create the ocata branch, so i can start to make simple cleanup PRs to it?

@apdibbo
Copy link
Contributor

apdibbo commented Aug 10, 2017

@stdweird I have pushed that now.

@stdweird
Copy link
Member Author

@apdibbo thanks!

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

No branches or pull requests

2 participants