Merged
Conversation
Contributor
There was a problem hiding this comment.
Pull Request Overview
This PR performs a vendor namespace change from RedberryProducts\MdNotion to Redberry\MdNotion across source, tests, configuration, examples, and documentation to align with a new vendor name.
- Updated all PHP namespaces, composer autoload mappings, service provider references, and tests.
- Updated documentation and example scripts to reflect the new namespace.
- Found a lingering class naming inconsistency and a documentation namespace error unrelated to just the vendor prefix change.
Reviewed Changes
Copilot reviewed 120 out of 120 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| tests/TestCase.php | Updated namespace and service provider reference. |
| tests/Services/PageReaderTest.php | Updated adapter and service namespaces. |
| tests/Services/IntegrationTest.php | Updated object and service namespaces. |
| tests/Services/DatabaseTableBasicTest.php | Updated Notion SDK and service namespaces. |
| tests/Services/DatabaseReaderTest.php | Updated object, SDK, and service namespaces. |
| tests/Services/BlockRegistryTest.php | Updated adapter references in tests. |
| tests/SDK/Requests/* | Updated SDK request namespaces. |
| tests/Pest.php | Updated TestCase namespace import. |
| tests/Objects/* | Updated object and service namespaces. |
| tests/MdNotionTest.php | Updated MdNotion core class references. |
| tests/FacadeTest.php | Updated Facade and service references. |
| tests/ContentBuilderTest.php | Updated ContentBuilder and service references. |
| tests/Adapters/* | Updated adapter namespaces in tests. |
| src/Traits/* | Updated trait namespaces and imports. |
| src/Services/* | Updated service namespaces and dependencies. |
| src/SDK/**/* | Updated SDK namespaces and request/resource imports. |
| src/Objects/* | Updated object namespaces and internal service resolution. |
| src/MdNotionServiceProvider.php | Updated provider namespace and bound classes. |
| src/MdNotion.php | Updated core class namespace and imports. |
| src/Facades/MdNotion.php | Updated facade namespace and target class reference. |
| src/DTOs/* | Updated DTO namespaces. |
| src/ContentBuilder.php | Updated builder namespace and service imports. |
| src/Commands/MdNotionCommand.php | Updated command namespace. |
| src/Adapters/* | Updated adapter namespaces and DTO imports; dynamic resolution lines affected. |
| src/Adapters/ColumnAdapter.php | Namespace updated; dynamic class resolution logic exposed. |
| config/md-notion.php | Updated config comment and adapter class FQCNs (contains a class name inconsistency). |
| composer.json | Updated PSR-4 autoload and provider/alias namespaces; keywords changed. |
| phpunit.xml.dist | Updated test suite name. |
| README.md | Updated usage examples to new namespace. |
| mdnotion_prd.md | Updated usage snippet (introduces an incorrect namespace). |
| examples/**/*.php | Updated example imports to new namespace. |
| index.php | Updated Notion SDK import. |
| column-list-fetch-example.php | Updated adapter and SDK imports. |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Contributor
There was a problem hiding this comment.
Pull Request Overview
Copilot reviewed 120 out of 120 changed files in this pull request and generated no new comments.
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This pull request updates the package namespace from
RedberryProducts\MdNotiontoRedberry\MdNotionacross the entire codebase and documentation. This change standardizes the naming to match the new package name and ensures consistency for autoloading, configuration, and usage examples. The update touches source files, configuration, documentation, and example scripts.Namespace and autoloading updates:
RedberryProducts\MdNotiontoRedberry\MdNotionthroughout source files, including adapters, DTOs, services, and facades. [1] [2] [3] [4] [5] [6] [7]composer.jsonto use the new namespace for PSR-4 autoloading and Laravel service provider/alias configuration. [1] [2]Configuration and adapter references:
config/md-notion.phpto use the new namespace.Documentation and example scripts:
README.md,mdnotion_prd.md) to use the new namespace in code samples and instructions. [1] [2] [3] [4] [5] [6]Other updates:
phpunit.xml.distto match the new package name.These changes ensure all references are consistent with the new package namespace, reducing confusion and potential autoloading issues.