[core] Deprecate LicenseInfo reexports

[cherniavskii]

Closes #11942

[mui-bot]

Deploy preview: https://deploy-preview-11956--material-ui-x.netlify.app/

Generated by 🚫 dangerJS against 3e64f35

[flaviendelangle]

Should we mention it in the migration guide or should we wait for its removal (in v8 I guess)?

[cherniavskii]

Should we mention it in the migration guide or should we wait for its removal (in v8 I guess)?

I don't think so, deprecation message should be enough for now

[oliviertassinari]

I wouldn't be against a runtime warning, but maybe it's pushing it too far 😄

[oliviertassinari]

This doesn't generate IntelliSense for me.

Actual:

How about?

Suggested change

	export {
	/**
	* @deprecated
	* Use `@mui/x-license` package instead:
	* import { LicenseInfo } from '@mui/x-license';
	*/
	LicenseInfo,
	} from '@mui/x-license';
	import { LicenseInfo as LicenseInfoExport } from '@mui/x-license';
	/**
	* @deprecated Use `@mui/x-license` package instead: import { LicenseInfo } from '@mui/x-license';
	*/
	export const LicenseInfo = LicenseInfoExport;

[cherniavskii]

Thanks, I couldn't figure it out!

[cherniavskii]

#11974

[flaviendelangle]

IMHO if we add a runtime warning, then we should add it properly on the migration guide because people would be almost forced to migrate (I don't want to stay with a warning in dev everytime I work on my app).

I see 2 ways of approaching this:

1. We do everything now: deprecate + runtime warning + migration guide
2. We only deprecate in the code, and in a few months we add a runtime warning

[oliviertassinari]

@flaviendelangle the smoother path makes sense, no need to rush it.