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

Modularise process_files.jl #1

Closed
tlienart opened this issue Jun 4, 2018 · 1 comment
Closed

Modularise process_files.jl #1

tlienart opened this issue Jun 4, 2018 · 1 comment

Comments

@tlienart
Copy link
Owner

tlienart commented Jun 4, 2018

The function is now too big and there are some parts that should be reasonably easy to cut out.

  1. the continuous checking part
  2. the markdown transformation part
  3. going through files and populating a dictionary accordingly

also really need to write some doc or think in more details about issues (currently all NOTE and TODO in the code).

  1. dealing with assets that are changing
  2. dealing with infra files that are changing and must cause a re-compile
  3. having some verbosity level indicating what's being seen and what's going on
@tlienart
Copy link
Owner Author

tlienart commented Jun 8, 2018

all done after the fixed merge in 1074d6e

@tlienart tlienart closed this as completed Jun 8, 2018
tlienart pushed a commit that referenced this issue Jan 27, 2020
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

No branches or pull requests

1 participant