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

[RFC] change directory structure #703

Closed

Conversation

IonBazan
Copy link
Member

@IonBazan IonBazan commented Sep 21, 2021

Moving all source files into src directory makes it easier to search trough the code since there is no need to exclude vendor dir all the time. Let me know what are your thoughts.

Warning: please note that Tests directory has been renamed to lowercase tests which might not be supported on some systems. You may need to rename it manually after checking out this branch.

@IonBazan IonBazan added the Idea label Sep 21, 2021
@IonBazan IonBazan force-pushed the feature/directory-structure branch 2 times, most recently from 1ff8a74 to 6082957 Compare September 21, 2021 06:23
@IonBazan IonBazan force-pushed the feature/directory-structure branch from 6082957 to 8456997 Compare September 21, 2021 06:24
@malarzm
Copy link
Member

malarzm commented Sep 21, 2021

@alcaeus I remember we were talking about this transition at least few times before, were we just procrastinating doing so or was there something different? I recall the argument about rebasing open PRs, but given there are just 4 other so it's not that important :)

@franmomu
Copy link
Contributor

I'm 👍, if it's because of the open PRs it not a big deal right now I think

@IonBazan IonBazan marked this pull request as draft October 1, 2021 12:42
@stale
Copy link

stale bot commented Apr 16, 2022

This issue has been automatically marked as stale because it has not had any recent activity. It will be closed in a week if no further activity occurs. Thank you for your contributions.

@stale stale bot added the Stale Issues that have not seen any activity in 30 days label Apr 16, 2022
@stale stale bot closed this Apr 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Idea Stale Issues that have not seen any activity in 30 days
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants