fix some comments (closes #3683)

evanw · evanw · commit dfa62069192b · 2024-03-14T13:03:57.000-04:00
diff --git a/internal/css_parser/css_nesting.go b/internal/css_parser/css_nesting.go
@@ -190,7 +190,7 @@ func (p *parser) lowerNestingInRuleWithContext(rule css_ast.Rule, context *lower
 			r.Selectors = []css_ast.ComplexSelector{merged}
 		}
 
-		// Pass 2: Substitue "&" for the parent selector
+		// Pass 2: Substitute "&" for the parent selector
 		if !p.options.unsupportedCSSFeatures.Has(compat.IsPseudoClass) || len(context.parentSelectors) <= 1 {
 			// If we can use ":is", or we don't have to because there's only one
 			// parent selector, or we are using ":is()" to match zero parent selectors
diff --git a/internal/js_parser/js_parser.go b/internal/js_parser/js_parser.go
@@ -8586,7 +8586,7 @@ func (p *parser) visitStmts(stmts []js_ast.Stmt, kind stmtsKind) []js_ast.Stmt {
 		for _, stmt := range before {
 			s, ok := stmt.Data.(*js_ast.SFunction)
 			if !ok {
-				// We may get non-function statements here in certain scenarious such as when "KeepNames" is enabled
+				// We may get non-function statements here in certain scenarios such as when "KeepNames" is enabled
 				nonFnStmts = append(nonFnStmts, stmt)
 				continue
 			}
diff --git a/internal/js_printer/js_printer.go b/internal/js_printer/js_printer.go
@@ -1545,7 +1545,7 @@ func (p *printer) printRequireOrImportExpr(importRecordIndex uint32, level js_as
 		defer p.printDotThenSuffix()
 	}
 
-	// Make sure the comma operator is propertly wrapped
+	// Make sure the comma operator is properly wrapped
 	if meta.ExportsRef != ast.InvalidRef && level >= js_ast.LComma {
 		p.print("(")
 		defer p.print(")")
diff --git a/internal/linker/linker.go b/internal/linker/linker.go
@@ -3370,7 +3370,7 @@ type cssImportOrder struct {
 // The only exception to this is "@layer". Evaluating a CSS file multiple
 // times is sort of equivalent to evaluating it once at the first location
 // as far as "@layer" is concerned. So we may in some cases keep both the
-// first and and last locations and only write out the "@layer" information
+// first and last locations and only write out the "@layer" information
 // for the first location.
 func (c *linkerContext) findImportedFilesInCSSOrder(entryPoints []uint32) (order []cssImportOrder) {
 	var visit func(uint32, []uint32, []css_ast.ImportConditions, []ast.ImportRecord)
@@ -3778,7 +3778,7 @@ func importConditionsAreEqual(a []css_ast.ImportConditions, b []css_ast.ImportCo
 // For "lib.css", the entry with the conditions [supports(display: flex)] should
 // make the entry with the conditions [supports(display: flex), screen] redundant.
 //
-// Note that all of this deliberately ignores the existance of "@layer" because
+// Note that all of this deliberately ignores the existence of "@layer" because
 // that is handled separately. All of this is only for handling unlayered styles.
 func isConditionalImportRedundant(earlier []css_ast.ImportConditions, later []css_ast.ImportConditions) bool {
 	if len(later) > len(earlier) {
diff --git a/internal/resolver/yarnpnp.go b/internal/resolver/yarnpnp.go
@@ -370,7 +370,7 @@ func (r resolverQuery) resolveViaFallback(manifest *pnpData, ident string) (pnpI
 	// Let referenceOrAlias be the entry from manifest.fallbackPool referenced by ident
 	referenceOrAlias, ok = manifest.fallbackPool[ident]
 
-	// Return it immediatly, whether it's defined or not
+	// Return it immediately, whether it's defined or not
 	if r.debugLogs != nil {
 		if ok {
 			r.debugLogs.addNote(fmt.Sprintf("    Found fallback for %q in \"fallbackPool\": [%s, %s]", ident,

Original file line number	Diff line number	Diff line change
`@@ -190,7 +190,7 @@ func (p parser) lowerNestingInRuleWithContext(rule css_ast.Rule, context lower`
`190`	`190`	`r.Selectors = []css_ast.ComplexSelector{merged}`
`191`	`191`	`}`
`192`	`192`
`193`		`- // Pass 2: Substitue "&" for the parent selector`
	`193`	`+ // Pass 2: Substitute "&" for the parent selector`
`194`	`194`	`if !p.options.unsupportedCSSFeatures.Has(compat.IsPseudoClass) \|\| len(context.parentSelectors) <= 1 {`
`195`	`195`	`// If we can use ":is", or we don't have to because there's only one`
`196`	`196`	`// parent selector, or we are using ":is()" to match zero parent selectors`
Original file line number	Diff line number	Diff line change
`@@ -8586,7 +8586,7 @@ func (p *parser) visitStmts(stmts []js_ast.Stmt, kind stmtsKind) []js_ast.Stmt {`
`8586`	`8586`	`for _, stmt := range before {`
`8587`	`8587`	`s, ok := stmt.Data.(*js_ast.SFunction)`
`8588`	`8588`	`if !ok {`
`8589`		`- // We may get non-function statements here in certain scenarious such as when "KeepNames" is enabled`
	`8589`	`+ // We may get non-function statements here in certain scenarios such as when "KeepNames" is enabled`
`8590`	`8590`	`nonFnStmts = append(nonFnStmts, stmt)`
`8591`	`8591`	`continue`
`8592`	`8592`	`}`
Original file line number	Diff line number	Diff line change
`@@ -1545,7 +1545,7 @@ func (p *printer) printRequireOrImportExpr(importRecordIndex uint32, level js_as`
`1545`	`1545`	`defer p.printDotThenSuffix()`
`1546`	`1546`	`}`
`1547`	`1547`
`1548`		`- // Make sure the comma operator is propertly wrapped`
	`1548`	`+ // Make sure the comma operator is properly wrapped`
`1549`	`1549`	`if meta.ExportsRef != ast.InvalidRef && level >= js_ast.LComma {`
`1550`	`1550`	`p.print("(")`
`1551`	`1551`	`defer p.print(")")`