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

Basic interactive prompts for init + refactoring #10

Merged
merged 7 commits into from
Jun 22, 2023
Merged

Basic interactive prompts for init + refactoring #10

merged 7 commits into from
Jun 22, 2023

Conversation

bhodrolok
Copy link
Owner

  • Now when running init, program prompts user thrice for setting up some values in the config.yml (base site URL, site description, and if they want to add custom templates)
    • Basic handling done so if user passes in nothing or just whitespaces, use default values.
  • Changed structure of creator.py, create_templates() --> 3 funcs for creating Jinja2 templates for home, note/posts and RSS feed. Same with create_content.
  • Also added function to create basic CSS for project upon init. ( Enhanced creator #5 )

… create_note_template(), setup for further refactoring

Improving code readability, maintainability and reusabilty!
…, update calls in create_project()

Also update function names to be snake_case as per naming standards(done in previous commit too)!
creator refactoring should be good for now
…default config.yml

looks good for setting up unit tests?
…ault config file's values

needs some ironing out
… to use defaults if invalid response(s)

Valid = Not None (empty) + Not just whitespaces.\nNow user can spam Enter!
@bhodrolok bhodrolok added the enhancement New feature or request label Jun 22, 2023
@bhodrolok bhodrolok self-assigned this Jun 22, 2023
@bhodrolok bhodrolok merged commit 1377815 into main Jun 22, 2023
@bhodrolok bhodrolok mentioned this pull request Nov 25, 2023
7 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

1 participant