Skip to content

[Fix] : resolve astro checks/CI recent failures#751

Open
Elchedli wants to merge 1 commit intosaicaca:mainfrom
Elchedli:fix/checkProblemsFix
Open

[Fix] : resolve astro checks/CI recent failures#751
Elchedli wants to merge 1 commit intosaicaca:mainfrom
Elchedli:fix/checkProblemsFix

Conversation

@Elchedli
Copy link
Copy Markdown

@Elchedli Elchedli commented Apr 8, 2026

Type of change

  • Bug fix (a non-breaking change that fixes an issue)
  • New feature (a non-breaking change that adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Other (please describe):

Checklist

  • I have read the CONTRIBUTING document.
  • I have checked to ensure that this Pull Request is not for personal changes.
  • I have performed a self-review of my own code.
  • My changes generate no new warnings.

Related Issue

#722
#750

Changes

Solved issues related to CI by fixing bugs in LightDarkSwitch and sortedPosts. Eliminated a declared unused variable to meet the criteria for astro check. This change will enable the GitHub Actions pipeline to be successful.

How To Test

  • Run pnpm check locally.
  • Alternatively, submit a PR and monitor the GitHub Actions failure.

Screenshots

0_errors
reason

The decision to downgrade @astrojs/svelte to version 7.2.0 (previous fuwari version) is a strategic move to prioritize build stability over dependency adoption. Upgrading would only be necessary if a newer version optimized the size of the injected hydration script or fixed a bug with how props are passed.

@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 8, 2026

@Elchedli is attempting to deploy a commit to the zephyirdgmailcom's projects Team on Vercel.

A member of the Team first needs to authorize it.

@JoeyC-Dev
Copy link
Copy Markdown
Contributor

JoeyC-Dev commented Apr 14, 2026

Generated the repository exactly from your fork, using the correct branch, but not working. Looks like you forget to generate the lockfile:

image

By the way, I tried to fix this issue in previous PR (#723), but it looks like the maintainer has already given up on maintaining this project. There are no official CI checks for this project anymore:
image

@Elchedli
Copy link
Copy Markdown
Author

Elchedli commented Apr 14, 2026

thank you @JoeyC-Dev for the insight, didn't know that the maintainer gave up on the project. Will fix the problem that you showed me. thank you for your help.

@CoderSerio
Copy link
Copy Markdown

experiencing the same issue, didn’t know the project was abandoned, that’s surprising 💔

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.

3 participants