Skip to content

Conversation

@jamos-tay
Copy link
Contributor

What is the purpose of this pull request? (put "X" next to an item, remove the rest)

• [X] Other, please explain: Change directory structure

Fixes #471

What is the rationale for this request?

Follow up from #467

What changes did you make? (Give an overview)

Revamp directory structure to use src instead of lib.

@jamos-tay jamos-tay changed the title Change lib to src Use src instead of lib directory Dec 13, 2018
@acjh
Copy link
Contributor

acjh commented Dec 13, 2018

lib/Page.jssrc/Page.js etc.

@jamos-tay
Copy link
Contributor Author

jamos-tay commented Dec 13, 2018

My bad, must've missed it, updated

@yamgent
Copy link
Member

yamgent commented Dec 14, 2018

As in the original comment, shouldn't it be src/lib/markbind/... rather than src/markbind/... ?

@jamos-tay jamos-tay force-pushed the move-files-to-src branch 2 times, most recently from f7bdaab to 9e9fa85 Compare December 14, 2018 09:58
@jamos-tay
Copy link
Contributor Author

Okay I made sure to double check it this time 😐

If possible maybe we can merge the PRs from the other guys in first, this PR will mess up their changes and I don't want to trouble them.

@yamgent
Copy link
Member

yamgent commented Dec 14, 2018

src/lib/markbind/package.json: The main: needs to be updated to reflect the new path.

@jamos-tay
Copy link
Contributor Author

Updated

@yamgent
Copy link
Member

yamgent commented Dec 18, 2018

@jamos-tay Need you to resolve conflict for this PR.

@jamos-tay
Copy link
Contributor Author

Sure, updated

@yamgent yamgent merged commit 57dd013 into MarkBind:master Dec 19, 2018
@yamgent yamgent added this to the v1.14.3 milestone Dec 19, 2018
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

Successfully merging this pull request may close these issues.

Use src instead of lib for directory structure

3 participants