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

Migrate package manager to backend #2024

Merged
merged 12 commits into from
Feb 16, 2021
Merged

Conversation

citizenmatt
Copy link
Member

This PR moves the code that calculates the list of in use packages from the frontend to the backend, allowing backend code to use the information. The data is passed to the fronted via the protocol, and is used to populate the Packages node of the Unity Explorer.

  • Packages are read from packages-lock.json if it exists and is up to date. If not, it will read/fall back to manifest.json. The packages-lock.json file is created and maintained automatically by Unity, and is not expected to be modified by hand. The user can modify manifest.json by hand, and Unity will read this file, resolve packages and update packages-lock.json. If Unity isn't running, the file will get out of date, so up to date packages are read from manifest.json
  • The Unity Explorer nodes have been refactored, mostly renaming and splitting into separate files
  • Any file or folder that the IDE treats as ignored is now highlighted in the Unity Explorer with a "hidden folder" icon, or a text marker to say that it is ignored. This is especially important for Unity projects as Unity ships code in Samples~ and docs in Documentation~ and the IDE ignores *~ by default, meaning these files will not appear in Find in Files or work if opened as a solution.
  • Reference nodes are no longer drawn using VCS status colours

@citizenmatt citizenmatt self-assigned this Feb 14, 2021
@citizenmatt citizenmatt added this to the Rider 2021.1 milestone Feb 14, 2021
@citizenmatt citizenmatt marked this pull request as ready for review February 16, 2021 12:04
@citizenmatt citizenmatt merged commit cb5c9ad into net211 Feb 16, 2021
@citizenmatt citizenmatt deleted the net211-mte-backend-packages branch February 16, 2021 13:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants