Conversation
🦋 Changeset detectedLatest commit: 657fd73 The changes in this PR will be included in the next version bump. This PR includes changesets to release 3 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
Coverage Report
File Coverage
|
||||||||||||||||||||||||||||||||||||||
|
| if (!componentsGroup[name]) { | ||
| ctx.report({ | ||
| message: `Two schemas are referenced with the same name but different content. Renamed ${prevName} to ${name}.`, | ||
| location: ctx.location, |
There was a problem hiding this comment.
Is there some reason for changing the report?
No location is needed anymore ?
There was a problem hiding this comment.
Or even no report enymore?
There was a problem hiding this comment.
The original file names are absolutely correct, and we are right to rename components during bundling too. So it doesn’t look like a bundler problem to me.
The only thing is that it won't show a specific location anymore, but it will still produce the warning in Terminal with the component names, so it's not a big deal to figure it out.
3 participants
What/Why/How?
Fixed an issue where schema name collision warnings appeared in bundle output when using stdout.
Reference
Addresses #661 (comment)
Testing
Screenshots (optional)
Check yourself
Security