Skip to content

ci: check issues’ states #12

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Jun 22, 2025
Merged

Conversation

YDX-2147483647
Copy link
Member

  • fix: add meta description for better SEO and clarity
  • ci: check issues’ states

Copy link

Diff between #12 and main

Feedback

Copy link

netlify bot commented Jun 22, 2025

Deploy Preview for clreq-gap-typst ready!

Name Link
🔨 Latest commit 3619f14
🔍 Latest deploy log https://app.netlify.com/projects/clreq-gap-typst/deploys/68578c9cb5b6cb00082f2893
😎 Deploy Preview https://deploy-preview-12--clreq-gap-typst.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR improves SEO by adding a meta description tag to the HTML template and enhances CI checks by updating issue state verification.

  • Added a meta description tag in typ/templates/template.html
  • Updated issue metadata and functions in typ/util.typ and scripts/check_issues.ts to include and validate the closed state of issues
  • Removed the deprecated scripts/check_uniq.ts and updated CI workflow and package.json accordingly

Reviewed Changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
typ/util.typ Added a new “closed” parameter to the issue function
typ/templates/template.html Added a meta description for improved SEO
scripts/check_uniq.ts Removed duplicate issues check script
scripts/check_issues.ts Added enhanced issue state checking logic
package.json Updated command to use the new check_issues script
main.typ Marked one duplicate issue as closed for clarity
.github/workflows/check.yml Added a new CI job for checking issue states
Comments suppressed due to low confidence (1)

@YDX-2147483647 YDX-2147483647 merged commit 591bd7c into typst-doc-cn:main Jun 22, 2025
8 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.

1 participant