Skip to content

fix: resolve JS and CSS lint errors across blocks#88

Merged
rofe merged 2 commits intomainfrom
fix-lint-errors
Apr 28, 2026
Merged

fix: resolve JS and CSS lint errors across blocks#88
rofe merged 2 commits intomainfrom
fix-lint-errors

Conversation

@rofe
Copy link
Copy Markdown
Collaborator

@rofe rofe commented Apr 28, 2026

Summary

  • article-header.js: remove trailing whitespace on 7 lines
  • featured-articles.js: replace for...of + await with Promise.all, add trailing newline
  • fluffyjaws-header.js: fix lines exceeding 100-char max-len, add trailing newline
  • tags.js: remove unused target variable, simplify to forEach
  • article-feed.css: modern color-function notation, shorthand fix
  • article-header.css: short hex colors, empty line before custom property
  • featured-article.css: modern color-function notation
  • styles.css: disable false-positive descending-specificity for theme overrides

Before: https://main--inside-aem--adobe.aem.page/
After: https://fix-lint-errors--inside-aem--adobe.aem.page/

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@aem-code-sync
Copy link
Copy Markdown

aem-code-sync Bot commented Apr 28, 2026

Hello, I'm the AEM Code Sync Bot and I will run some actions to deploy your branch.
In case there are problems, just click the checkbox below to rerun the respective action.

  • Re-sync branch

Note: PSI checks are disabled due to site authentication. Please utilize your org/client side GHA PSI implementation

Commits

@rofe rofe requested a review from mtilburgadobe April 28, 2026 10:02
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@rofe rofe changed the title fix: resolve JS lint errors across blocks fix: resolve JS and CSS lint errors across blocks Apr 28, 2026
@rofe rofe merged commit c0abd5d into main Apr 28, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants