Skip to content

Astro v5#1366

Merged
jtamatsukuri merged 4 commits intomasterfrom
202501-astro-v5
May 31, 2025
Merged

Astro v5#1366
jtamatsukuri merged 4 commits intomasterfrom
202501-astro-v5

Conversation

@haxibami
Copy link
Copy Markdown
Member

@haxibami haxibami commented Jan 17, 2025

一旦最小限の変更でトライしています

影響が大きそうな点

  • .mdx ファイル内で <Fragment /> が使えなくなったようです
    • 代わりに <></> を使ってください
  • Sassの @import はdeprecatedになりました
    • 代わりに @use@forward をうまく使ってください
    • 変更を最小限にしたい場合、@use foo.scss as *; のようにするとよいかもしれません

今後別PRで修正する点

  • Sassの@importを使っている箇所を@useに置き換える
  • Astro.globを使っている箇所をコンテンツコレクションに置き換える

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jan 17, 2025

🚀 Deployed on https://deploy-preview-1366--utelecon.netlify.app

@haxibami
Copy link
Copy Markdown
Member Author

アーッ
image

@haxibami
Copy link
Copy Markdown
Member Author

haxibami commented Jan 17, 2025

#1217 を先に済ませることとします 代わりに #1367 が立ちました

@haxibami haxibami marked this pull request as ready for review February 24, 2025 06:48
Copy link
Copy Markdown
Member

@seieric seieric left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

動作確認を中心にレビューしました。一部の警告は今後の課題ということで無視しました。
package.jsonpackage-lock.jsonがconflictしているのでそこだけ修正いただきたいです。

Copy link
Copy Markdown
Member

@seieric seieric left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

エラー発生時にエラーハンドラ内でエラーが発生して、原因となるエラーが正しく表示されないようです。

@seieric seieric self-requested a review May 31, 2025 04:22
@jtamatsukuri jtamatsukuri merged commit 1b986bf into master May 31, 2025
2 checks passed
@jtamatsukuri jtamatsukuri deleted the 202501-astro-v5 branch May 31, 2025 05:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants