Add missing .meta for DCLPulse.Transport.Shared.csproj - #36
Merged
Conversation
When this folder is consumed as an immutable Git UPM package, Unity requires a .meta for every file. The csproj (kept so the package also builds standalone with `dotnet build`) had none, producing: Asset .../DCLPulse.Transport.Shared.csproj has no meta file, but it's in an immutable folder. The asset will be ignored. Add a DefaultImporter .meta so Unity imports it as a harmless text asset and stops warning. Non-breaking: `dotnet build` already ignores *.meta via the existing `<None Remove="**/*.meta" />`. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Contributor
Author
|
Companion consumer PR: decentraland/unity-explorer#9669 (draft, blocked on this merge). |
mikhail-dcl
approved these changes
Aug 10, 2026
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
2 participants
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.
Problem
When
src/DCLPulse.Transport.Sharedis consumed as an immutable Git UPM package (e.g. fromunity-explorer'smanifest.json), Unity requires a.metafile for every file in the package. Every file has one exceptDCLPulse.Transport.Shared.csproj, so Unity logs on every import:The csproj is intentionally kept in the package folder so it can also be built standalone with
dotnet build— it just never got a.meta.Fix
Add a standard
DefaultImporter.metafor the csproj. Unity then imports it as a harmless text asset and the warning disappears.Non-breaking:
dotnet buildstill ignores*.metavia the existing<None Remove="**/*.meta" />in the csproj, so the standalone build is unaffected.Follow-up
Once merged,
decentraland/unity-explorerbumps itscom.decentraland.pulse.transportpin to the merge commit — see the companion PR.🤖 Generated with Claude Code