Open
Conversation
There was a problem hiding this comment.
Pull Request Overview
This pull request introduces a new Unity Package Installer containing the complete Asset Store Tools package including uploader, validator, and preview generation capabilities. The PR replaces Unity CI workflow files with a comprehensive Unity Editor package.
- Introduces complete Asset Store Tools package with uploader, validator, and preview generator components
- Adds UI framework with UIElements-based interfaces for package management
- Implements comprehensive API layer for Asset Store communication and package uploading
Reviewed Changes
Copilot reviewed 298 out of 1159 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| Installer/Packages/com.unity.asset-store-tools/Editor/Previews/ | Comprehensive preview generation system including UI elements, data models, generators, and utility classes |
| Installer/Packages/com.unity.asset-store-tools/Editor/Exporter/ | Package export functionality with abstraction layer and multiple export implementations |
| Installer/Packages/com.unity.asset-store-tools/Editor/Api/ | Asset Store API integration including authentication, uploading, and response handling |
| Multiple .meta files | Unity asset metadata files for proper package integration |
Comments suppressed due to low confidence (1)
Installer/Packages/com.unity.asset-store-tools/Editor/Previews/Scripts/UI/Elements/PreviewGeneratorPathsElement.cs:1
- The error message refers to 'validation path' but this is in the context of preview generation paths. The message should refer to 'preview generation path' for consistency and clarity.
using AssetStoreTools.Previews.UI.Data;
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
...-tools/Editor/Previews/Scripts/Generators/Custom/TypeGenerators/AudioTypePreviewGenerator.cs
Show resolved
Hide resolved
...ler/Packages/com.unity.asset-store-tools/Editor/Exporter/Abstractions/PackageExporterBase.cs
Show resolved
Hide resolved
Removed OpenUPM CLI installation steps from both README files and updated the installer download link in Unity-Package/Assets/root/README.md. Changed the default solution in VSCode settings to Unity-Package.sln for improved project configuration.
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 removes all workflow YAML files for Unity CI builds and tests across multiple Unity versions. These files previously defined GitHub Actions workflows for building, running edit mode and play mode tests for various Unity releases. With these deletions, automated CI for Unity projects via GitHub Actions is disabled for all listed versions.
Removal of Unity CI workflows
2019.4.40f1,2020.3.40f1,2021.3.45f1,2022.3.57f1,2023.1.20f1,2023.2.20f1, and6000.0.37f1(.github/workflows/*_build.yml). [1] [2] [3] [4] [5] [6] [7].github/workflows/*_editor.yml). [1] [2] [3] [4] [5] [6] [7].github/workflows/*_player.yml). [1] [2] [3] [4] [5] [6]This change disables all automated build and test jobs for Unity projects that were previously triggered by pushes or pull requests to the repository.