Skip to content

Add is:inline directive to external script tags#19

Merged
goude merged 1 commit intomainfrom
claude/fix-linting-upgrade-astro-CGVot
Apr 2, 2026
Merged

Add is:inline directive to external script tags#19
goude merged 1 commit intomainfrom
claude/fix-linting-upgrade-astro-CGVot

Conversation

@goude
Copy link
Copy Markdown
Owner

@goude goude commented Apr 2, 2026

Summary

Added the is:inline directive to three external script tags that load third-party libraries (WaveSurfer.js, WaveSurfer regions plugin, and OpenSheetMusicDisplay).

Changes

  • Added is:inline directive to the WaveSurfer.js library script tag
  • Added is:inline directive to the WaveSurfer.js regions plugin script tag
  • Added is:inline directive to the OpenSheetMusicDisplay library script tag
  • Updated Astro dependency from ^5.17.3 to ^6.1.3

Details

The is:inline directive ensures these external scripts are inlined in the HTML output rather than being processed through Astro's module system. This is appropriate for third-party CDN-hosted libraries that are already optimized and should be loaded as-is without additional bundling or transformation.

https://claude.ai/code/session_01Tek4Bbs9j1izXAfMJ9awfH

- Add is:inline to external script tags in do-olls page to silence
  astro(4000) hints about unprocessed scripts with attributes
- Upgrade astro from ^5.17.3 to 6.1.3 (zero warnings, build passes)

https://claude.ai/code/session_01Tek4Bbs9j1izXAfMJ9awfH
@goude goude merged commit f90a00d into main Apr 2, 2026
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