Merge pull request #6163 from NomicFoundation/small-comments-improvements

Improve two build-system related comments

Author: alcuadrado

v-next/hardhat/src/internal/builtin-plugins/solidity/build-system/build-system.ts

@@ -263,8 +263,9 @@ export class SolidityBuildSystemImplementation implements SolidityBuildSystem {
         DependencyGraphImplementation
       > = new Map();
 
-      // Note: This is merging solc config object identity, not a deep
-      // comparison. This is Hardhat v2's behavior.
+      // Note: This groups the subgraphs by solc config. It compares the configs
+      // based on reference, and not by deep equality. It misses some merging
+      // opportunities, but this is Hardhat v2's behavior and works well enough.
       for (const [solcConfig, subgraph] of subgraphsWithConfig) {
         const mergedSubgraph = mergedSubgraphsByConfig.get(solcConfig);
 

v-next/hardhat/src/internal/builtin-plugins/solidity/build-system/resolver/dependency-resolver.ts

@@ -52,7 +52,7 @@ import {
 //    packages, by forbidding user remappings with context and starting with
 //    `npm/`.
 //  - We do allow users to remap imports of non-npm files, including remmapping
-//    them into file within npm packages.
+//    them into npm packages.
 //  - Every import in an npm file is either relative or remapped by a remapping
 //    generated by the resolver.
 //  - A direct import (i.e. not relative) is considered to be local within the