feature symfony#51247 [AssetMapper] Mark component as non experimental (fabpot)

fabpot · fabpot · commit 7e4793b281e7 · 2023-08-03T12:58:09.000+02:00
This PR was merged into the 6.4 branch. Discussion ---------- [AssetMapper] Mark component as non experimental | Q | A | ------------- | --- | Branch? | 6.4 | Bug fix? | no | New feature? | yes  | Deprecations? | no  | Tickets | n/a | License | MIT | Doc PR | n/a No feature can be experimental in 6.4. `@weaverryan` Commits ------- 530406a [AssetMapper] Mark component as non experimental
diff --git a/src/Symfony/Component/AssetMapper/AssetDependency.php b/src/Symfony/Component/AssetMapper/AssetDependency.php
@@ -13,8 +13,6 @@
 
 /**
  * Represents a dependency that a MappedAsset has.
- *
- * @experimental
  */
 final class AssetDependency
 {
diff --git a/src/Symfony/Component/AssetMapper/AssetMapper.php b/src/Symfony/Component/AssetMapper/AssetMapper.php
@@ -17,8 +17,6 @@
 /**
  * Finds and returns assets in the pipeline.
  *
- * @experimental
- *
  * @final
  */
 class AssetMapper implements AssetMapperInterface
diff --git a/src/Symfony/Component/AssetMapper/AssetMapperCompiler.php b/src/Symfony/Component/AssetMapper/AssetMapperCompiler.php
@@ -16,8 +16,6 @@
 /**
  * Runs a chain of compiles intended to adjust the source of assets.
  *
- * @experimental
- *
  * @final
  */
 class AssetMapperCompiler
diff --git a/src/Symfony/Component/AssetMapper/AssetMapperDevServerSubscriber.php b/src/Symfony/Component/AssetMapper/AssetMapperDevServerSubscriber.php
@@ -21,8 +21,6 @@
 /**
  * Functions like a controller that returns assets from the asset mapper.
  *
- * @experimental
- *
  * @author Ryan Weaver <ryan@symfonycasts.com>
  */
 final class AssetMapperDevServerSubscriber implements EventSubscriberInterface
diff --git a/src/Symfony/Component/AssetMapper/AssetMapperInterface.php b/src/Symfony/Component/AssetMapper/AssetMapperInterface.php
@@ -14,8 +14,6 @@
 /**
  * Finds and returns assets in the pipeline.
  *
- * @experimental
- *
  * @author Ryan Weaver <ryan@symfonycasts.com>
  */
 interface AssetMapperInterface
diff --git a/src/Symfony/Component/AssetMapper/AssetMapperRepository.php b/src/Symfony/Component/AssetMapper/AssetMapperRepository.php
@@ -17,8 +17,6 @@
 /**
  * Finds assets in the asset mapper.
  *
- * @experimental
- *
  * @author Ryan Weaver <ryan@symfonycasts.com>
  *
  * @final
diff --git a/src/Symfony/Component/AssetMapper/CHANGELOG.md b/src/Symfony/Component/AssetMapper/CHANGELOG.md
@@ -1,6 +1,11 @@
 CHANGELOG
 =========
 
+6.4
+---
+
+ * Mark the component as non experimental
+
 6.3
 ---
 
diff --git a/src/Symfony/Component/AssetMapper/Command/AssetMapperCompileCommand.php b/src/Symfony/Component/AssetMapper/Command/AssetMapperCompileCommand.php
@@ -28,8 +28,6 @@
  *
  * This command is intended to be used during deployment.
  *
- * @experimental
- *
  * @author Ryan Weaver <ryan@symfonycasts.com>
  */
 #[AsCommand(name: 'asset-map:compile', description: 'Compiles all mapped assets and writes them to the final public output directory.')]
diff --git a/src/Symfony/Component/AssetMapper/Command/DebugAssetMapperCommand.php b/src/Symfony/Component/AssetMapper/Command/DebugAssetMapperCommand.php
@@ -22,8 +22,6 @@
 /**
  * Outputs all the assets in the asset mapper.
  *
- * @experimental
- *
  * @author Ryan Weaver <ryan@symfonycasts.com>
  */
 #[AsCommand(name: 'debug:asset-map', description: 'Outputs all mapped assets.')]
diff --git a/src/Symfony/Component/AssetMapper/Command/ImportMapExportCommand.php b/src/Symfony/Component/AssetMapper/Command/ImportMapExportCommand.php
@@ -18,8 +18,6 @@
 use Symfony\Component\Console\Output\OutputInterface;
 
 /**
- * @experimental
- *
  * @author Kévin Dunglas <kevin@dunglas.dev>
  */
 #[AsCommand(name: 'importmap:export', description: 'Exports the importmap JSON')]
diff --git a/src/Symfony/Component/AssetMapper/Command/ImportMapRemoveCommand.php b/src/Symfony/Component/AssetMapper/Command/ImportMapRemoveCommand.php
@@ -20,8 +20,6 @@
 use Symfony\Component\Console\Style\SymfonyStyle;
 
 /**
- * @experimental
- *
  * @author Kévin Dunglas <kevin@dunglas.dev>
  */
 #[AsCommand(name: 'importmap:remove', description: 'Removes JavaScript packages')]
diff --git a/src/Symfony/Component/AssetMapper/Command/ImportMapRequireCommand.php b/src/Symfony/Component/AssetMapper/Command/ImportMapRequireCommand.php
@@ -25,8 +25,6 @@
 use Symfony\Component\Console\Style\SymfonyStyle;
 
 /**
- * @experimental
- *
  * @author Kévin Dunglas <kevin@dunglas.dev>
  */
 #[AsCommand(name: 'importmap:require', description: 'Requires JavaScript packages')]
diff --git a/src/Symfony/Component/AssetMapper/Command/ImportMapUpdateCommand.php b/src/Symfony/Component/AssetMapper/Command/ImportMapUpdateCommand.php
@@ -19,8 +19,6 @@
 use Symfony\Component\Console\Style\SymfonyStyle;
 
 /**
- * @experimental
- *
  * @author Kévin Dunglas <kevin@dunglas.dev>
  */
 #[AsCommand(name: 'importmap:update', description: 'Updates all JavaScript packages to their latest versions')]
diff --git a/src/Symfony/Component/AssetMapper/Compiler/AssetCompilerInterface.php b/src/Symfony/Component/AssetMapper/Compiler/AssetCompilerInterface.php
@@ -17,8 +17,6 @@
 /**
  * An asset compiler is responsible for applying any changes to the contents of an asset.
  *
- * @experimental
- *
  * @author Ryan Weaver <ryan@symfonycasts.com>
  */
 interface AssetCompilerInterface
diff --git a/src/Symfony/Component/AssetMapper/Compiler/AssetCompilerPathResolverTrait.php b/src/Symfony/Component/AssetMapper/Compiler/AssetCompilerPathResolverTrait.php
@@ -16,8 +16,6 @@
 /**
  * Helps resolve "../" and "./" in paths.
  *
- * @experimental
- *
  * @internal
  */
 trait AssetCompilerPathResolverTrait
diff --git a/src/Symfony/Component/AssetMapper/Compiler/CssAssetUrlCompiler.php b/src/Symfony/Component/AssetMapper/Compiler/CssAssetUrlCompiler.php
@@ -21,8 +21,6 @@
  * Resolves url() paths in CSS files.
  *
  * Originally sourced from https://github.com/rails/propshaft/blob/main/lib/propshaft/compilers/css_asset_urls.rb
- *
- * @experimental
  */
 final class CssAssetUrlCompiler implements AssetCompilerInterface
 {
diff --git a/src/Symfony/Component/AssetMapper/Compiler/JavaScriptImportPathCompiler.php b/src/Symfony/Component/AssetMapper/Compiler/JavaScriptImportPathCompiler.php
@@ -20,8 +20,6 @@
 /**
  * Resolves import paths in JS files.
  *
- * @experimental
- *
  * @author Ryan Weaver <ryan@symfonycasts.com>
  */
 final class JavaScriptImportPathCompiler implements AssetCompilerInterface
diff --git a/src/Symfony/Component/AssetMapper/Compiler/SourceMappingUrlsCompiler.php b/src/Symfony/Component/AssetMapper/Compiler/SourceMappingUrlsCompiler.php
@@ -19,8 +19,6 @@
  * Rewrites already-existing source map URLs to their final digested path.
  *
  * Originally sourced from https://github.com/rails/propshaft/blob/main/lib/propshaft/compilers/source_mapping_urls.rb
- *
- * @experimental
  */
 final class SourceMappingUrlsCompiler implements AssetCompilerInterface
 {
diff --git a/src/Symfony/Component/AssetMapper/ImportMap/ImportMapEntry.php b/src/Symfony/Component/AssetMapper/ImportMap/ImportMapEntry.php
@@ -14,8 +14,6 @@
 /**
  * Represents an item that should be in the importmap.
  *
- * @experimental
- *
  * @author Ryan Weaver <ryan@symfonycasts.com>
  */
 final class ImportMapEntry
diff --git a/src/Symfony/Component/AssetMapper/ImportMap/ImportMapManager.php b/src/Symfony/Component/AssetMapper/ImportMap/ImportMapManager.php
@@ -18,8 +18,6 @@
 use Symfony\Component\VarExporter\VarExporter;
 
 /**
- * @experimental
- *
  * @author Kévin Dunglas <kevin@dunglas.dev>
  * @author Ryan Weaver <ryan@symfonycasts.com>
  *
diff --git a/src/Symfony/Component/AssetMapper/ImportMap/ImportMapRenderer.php b/src/Symfony/Component/AssetMapper/ImportMap/ImportMapRenderer.php
@@ -12,8 +12,6 @@
 namespace Symfony\Component\AssetMapper\ImportMap;
 
 /**
- * @experimental
- *
  * @author Kévin Dunglas <kevin@dunglas.dev>
  * @author Ryan Weaver <ryan@symfonycasts.com>
  *
diff --git a/src/Symfony/Component/AssetMapper/ImportMap/PackageRequireOptions.php b/src/Symfony/Component/AssetMapper/ImportMap/PackageRequireOptions.php
@@ -14,8 +14,6 @@
 /**
  * Represents a package that should be installed or updated.
  *
- * @experimental
- *
  * @author Kévin Dunglas <kevin@dunglas.dev>
  */
 final class PackageRequireOptions
diff --git a/src/Symfony/Component/AssetMapper/ImportMap/Resolver/JsDelivrEsmResolver.php b/src/Symfony/Component/AssetMapper/ImportMap/Resolver/JsDelivrEsmResolver.php
@@ -17,9 +17,6 @@
 use Symfony\Contracts\HttpClient\Exception\HttpExceptionInterface;
 use Symfony\Contracts\HttpClient\HttpClientInterface;
 
-/**
- * @experimental
- */
 final class JsDelivrEsmResolver implements PackageResolverInterface
 {
     public const URL_PATTERN_VERSION = 'https://data.jsdelivr.com/v1/packages/npm/%s/resolved?specifier=%s';
diff --git a/src/Symfony/Component/AssetMapper/ImportMap/Resolver/JspmResolver.php b/src/Symfony/Component/AssetMapper/ImportMap/Resolver/JspmResolver.php
@@ -17,9 +17,6 @@
 use Symfony\Component\HttpClient\HttpClient;
 use Symfony\Contracts\HttpClient\HttpClientInterface;
 
-/**
- * @experimental
- */
 final class JspmResolver implements PackageResolverInterface
 {
     public const BASE_URI = 'https://api.jspm.io/';
diff --git a/src/Symfony/Component/AssetMapper/ImportMap/Resolver/PackageResolver.php b/src/Symfony/Component/AssetMapper/ImportMap/Resolver/PackageResolver.php
@@ -13,9 +13,6 @@
 
 use Psr\Container\ContainerInterface;
 
-/**
- * @experimental
- */
 final class PackageResolver implements PackageResolverInterface
 {
     public function __construct(
diff --git a/src/Symfony/Component/AssetMapper/ImportMap/Resolver/PackageResolverInterface.php b/src/Symfony/Component/AssetMapper/ImportMap/Resolver/PackageResolverInterface.php
@@ -13,9 +13,6 @@
 
 use Symfony\Component\AssetMapper\ImportMap\PackageRequireOptions;
 
-/**
- * @experimental
- */
 interface PackageResolverInterface
 {
     /**
diff --git a/src/Symfony/Component/AssetMapper/ImportMap/Resolver/ResolvedImportMapPackage.php b/src/Symfony/Component/AssetMapper/ImportMap/Resolver/ResolvedImportMapPackage.php
@@ -13,9 +13,6 @@
 
 use Symfony\Component\AssetMapper\ImportMap\PackageRequireOptions;
 
-/**
- * @experimental
- */
 final class ResolvedImportMapPackage
 {
     public function __construct(
diff --git a/src/Symfony/Component/AssetMapper/MappedAsset.php b/src/Symfony/Component/AssetMapper/MappedAsset.php
@@ -14,8 +14,6 @@
 /**
  * Represents a single asset in the asset mapper system.
  *
- * @experimental
- *
  * @author Ryan Weaver <ryan@symfonycasts.com>
  */
 final class MappedAsset
diff --git a/src/Symfony/Component/AssetMapper/MapperAwareAssetPackage.php b/src/Symfony/Component/AssetMapper/MapperAwareAssetPackage.php
@@ -16,8 +16,6 @@
 /**
  * Decorates asset packages to support resolving assets from the asset mapper.
  *
- * @experimental
- *
  * @author Ryan Weaver <ryan@symfonycasts.com>
  */
 final class MapperAwareAssetPackage implements PackageInterface
diff --git a/src/Symfony/Component/AssetMapper/README.md b/src/Symfony/Component/AssetMapper/README.md
@@ -6,11 +6,6 @@ then moved to a public directory with digested (i.e. versioned) filenames. It
 also allows you to dump an [importmap](https://developer.mozilla.org/en-US/docs/Web/HTML/Element/script/type/importmap)
 to allow writing modern JavaScript without a build system.
 
-**This Component is experimental**.
-[Experimental features](https://symfony.com/doc/current/contributing/code/experimental.html)
-are not covered by Symfony's
-[Backward Compatibility Promise](https://symfony.com/doc/current/contributing/code/bc.html).
-
 Resources
 ---------
 

Original file line number	Diff line number	Diff line change
`@@ -13,8 +13,6 @@`
`13`	`13`
`14`	`14`	`/**`
`15`	`15`	`* Represents a dependency that a MappedAsset has.`
`16`		`- *`
`17`		`- * @experimental`
`18`	`16`	`*/`
`19`	`17`	`final class AssetDependency`
`20`	`18`	`{`
Original file line number	Diff line number	Diff line change
`@@ -17,8 +17,6 @@`
`17`	`17`	`/**`
`18`	`18`	`* Finds and returns assets in the pipeline.`
`19`	`19`	`*`
`20`		`- * @experimental`
`21`		`- *`
`22`	`20`	`* @final`
`23`	`21`	`*/`
`24`	`22`	`class AssetMapper implements AssetMapperInterface`
Original file line number	Diff line number	Diff line change
`@@ -16,8 +16,6 @@`
`16`	`16`	`/**`
`17`	`17`	`* Runs a chain of compiles intended to adjust the source of assets.`
`18`	`18`	`*`
`19`		`- * @experimental`
`20`		`- *`
`21`	`19`	`* @final`
`22`	`20`	`*/`
`23`	`21`	`class AssetMapperCompiler`
Original file line number	Diff line number	Diff line change
`@@ -21,8 +21,6 @@`
`21`	`21`	`/**`
`22`	`22`	`* Functions like a controller that returns assets from the asset mapper.`
`23`	`23`	`*`
`24`		`- * @experimental`
`25`		`- *`
`26`	`24`	`* @author Ryan Weaver <[email protected]>`
`27`	`25`	`*/`
`28`	`26`	`final class AssetMapperDevServerSubscriber implements EventSubscriberInterface`
Original file line number	Diff line number	Diff line change
`@@ -14,8 +14,6 @@`
`14`	`14`	`/**`
`15`	`15`	`* Finds and returns assets in the pipeline.`
`16`	`16`	`*`
`17`		`- * @experimental`
`18`		`- *`
`19`	`17`	`* @author Ryan Weaver <[email protected]>`
`20`	`18`	`*/`
`21`	`19`	`interface AssetMapperInterface`
Original file line number	Diff line number	Diff line change
`@@ -28,8 +28,6 @@`
`28`	`28`	`*`
`29`	`29`	`* This command is intended to be used during deployment.`
`30`	`30`	`*`
`31`		`- * @experimental`
`32`		`- *`
`33`	`31`	`* @author Ryan Weaver <[email protected]>`
`34`	`32`	`*/`
`35`	`33`	`#[AsCommand(name: 'asset-map:compile', description: 'Compiles all mapped assets and writes them to the final public output directory.')]`
Original file line number	Diff line number	Diff line change
`@@ -22,8 +22,6 @@`
`22`	`22`	`/**`
`23`	`23`	`* Outputs all the assets in the asset mapper.`
`24`	`24`	`*`
`25`		`- * @experimental`
`26`		`- *`
`27`	`25`	`* @author Ryan Weaver <[email protected]>`
`28`	`26`	`*/`
`29`	`27`	`#[AsCommand(name: 'debug:asset-map', description: 'Outputs all mapped assets.')]`