michael-s-molina opened a new pull request, #38476:
URL: https://github.com/apache/superset/pull/38476

   ### SUMMARY
   
   Removes the redundant `superset_extensions` intermediate directory from the 
backend source tree of extensions. The publisher name already identifies the 
code as coming from an external source.
   
   **Before:**
   ```
   backend/src/
   └── superset_extensions/
       └── my_org/
           └── my_extension/
               └── entrypoint.py
   ```
   
   **After:**
   ```
   backend/src/
   └── my_org/
       └── my_extension/
           └── entrypoint.py
   ```
   
   The generated entrypoint module path changes accordingly:
   
   | | Before | After |
   |---|---|---|
   | Folder structure | `backend/src/superset_extensions/{publisher}/{name}/` | 
`backend/src/{publisher}/{name}/` |
   | Python entrypoint | `superset_extensions.my_org.my_extension.entrypoint` | 
`my_org.my_extension.entrypoint` |
   | pyproject.toml include | 
`src/superset_extensions/my_org/my_extension/**/*.py` | 
`src/my_org/my_extension/**/*.py` |
   
   ### TESTING INSTRUCTIONS
   1. Scaffold a new extension and verify the generated structure:
      ```bash
      superset-extensions init
      # publisher: my-org, name: my-ext, include backend: yes
      ls my-org.my-ext/backend/src/
      # Expected: my_org/  (no superset_extensions/ prefix)
      ```
   
   2. Run `superset-extensions validate` inside the scaffolded extension — 
should pass.
   
   3. Run `superset-extensions build` — `dist/manifest.json` should contain:
      ```json
      "entrypoint": "my_org.my_ext.entrypoint"
      ```
   
   4. Run the CLI test suite:
      ```bash
      cd superset-extensions-cli && pytest
      ```
   
   5. Run host-side unit tests:
      ```bash
      pytest tests/unit_tests/extensions/
      ```
   
   ### ADDITIONAL INFORMATION
   <!--- Check any relevant boxes with "x" -->
   <!--- HINT: Include "Fixes #nnn" if you are fixing an existing issue -->
   - [ ] Has associated issue:
   - [ ] Required feature flags:
   - [ ] Changes UI
   - [ ] Includes DB Migration (follow approval process in 
[SIP-59](https://github.com/apache/superset/issues/13351))
     - [ ] Migration is atomic, supports rollback & is backwards-compatible
     - [ ] Confirm DB migration upgrade and downgrade tested
     - [ ] Runtime estimates and downtime expectations provided
   - [ ] Introduces new feature or API
   - [ ] Removes existing feature or API
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to