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

Moved utils to a new folder #348

Closed
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
3 participants
@mikicz
Copy link
Member

mikicz commented Mar 16, 2018

This PR is also preparing the ground for bigger changes. In the thesis, the code grows quite a bit and the naucse folder is becoming crowded. I think this move will improve code orientation. Furthermore, it also removes the inconsistency of naming the files (modelutils.py vs markdown_util.py).

Feel free to close the PR if you don't like the change.

@hroncok hroncok requested a review from encukou Mar 16, 2018

@hroncok

This comment has been minimized.

Copy link
Member

hroncok commented Mar 16, 2018

I'm OK with this change. Let's wait for @encukou.

@encukou encukou closed this in 55e4c89 Mar 17, 2018

@encukou

This comment has been minimized.

Copy link
Member

encukou commented Mar 17, 2018

Looks reasonable, thanks. Rebased & merged.

@mikicz mikicz deleted the mikicz:utils-folder branch Mar 17, 2018

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