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

Switch to a modern directory structure #535

Merged
merged 1 commit into from Mar 27, 2024

Conversation

greg0ire
Copy link
Member

@greg0ire greg0ire commented Mar 26, 2024

Here are the steps I followed (probably not an exhaustive list, but can be useful to migrate more bundles):

  • mkdir src
  • git mv Collector DependencyInjection DoctrineMigrationsBundle.php MigrationsFactory src
  • adjust autoloading path
  • tweak PHPUnit config
  • tweak Psalm config
  • tweak phpcs config
  • git mv Resources/config config
  • git mv Resources/docs doc
  • overrode Bundle::getPath()
  • git mv Tests tests
  • git mv Resources/views templates

Here are the steps I followed:

- `mkdir src`
- `git mv Collector DependencyInjection DoctrineMigrationsBundle.php MigrationsFactory src`
- adjust autoloading path
- tweak PHPUnit config
- tweak Psalm config
- `git mv Resources/config config`
- `git mv Resources/docs doc`
- overrode `Bundle::getPath()`
- `git mv Tests tests`
- `git mv Resources/views templates`
@greg0ire greg0ire merged commit dcb2981 into doctrine:3.4.x Mar 27, 2024
15 checks passed
@greg0ire greg0ire deleted the modern-directory-structure branch March 27, 2024 06:34
@greg0ire greg0ire added this to the 3.4.0 milestone Mar 27, 2024
@connorhu
Copy link

connorhu commented Apr 8, 2024

@greg0ire +1 item to the todo list: update .gitattributes
I will do an archive from 3.4.x and check out what needs to be updated and I'll open a PR.

@greg0ire
Copy link
Member Author

greg0ire commented Apr 8, 2024

Ah indeed, I missed at least Tests

@connorhu connorhu mentioned this pull request Apr 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants