Skip to content

fix: Vale linter, npm prettier in dev container and CI workflow#167

Merged
BrianDangDev merged 5 commits intomainfrom
development
Feb 2, 2026
Merged

fix: Vale linter, npm prettier in dev container and CI workflow#167
BrianDangDev merged 5 commits intomainfrom
development

Conversation

@SteveDala
Copy link
Collaborator

Description

This change ensures that Vale is actually running when people use npm run prose:check and that it is part of the CI workflow (at this stage, just to check for errors in the prose)

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • Documentation (update or new)

How Has This Been Tested?

Tested workflow with a pass on the development branch.

Testing Checklist

  • Have run npm run format
  • Have run npm run build
  • Have run npm run dev and/or npm run preview, using:
    • Chrome
    • Safari
    • Firefox

Checklist

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have requested a review from @BrianDangDev on the Pull Request

@netlify
Copy link

netlify bot commented Feb 1, 2026

Deploy Preview for thoth-tech ready!

Name Link
🔨 Latest commit 72b4123
🔍 Latest deploy log https://app.netlify.com/projects/thoth-tech/deploys/697ea1592aebc00008f86023
😎 Deploy Preview https://deploy-preview-167--thoth-tech.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.

@SteveDala
Copy link
Collaborator Author

@BrianDangDev would appreciate a review from you. 😄

BDang-ctrl

This comment was marked as duplicate.

Copy link
Contributor

@BrianDangDev BrianDangDev left a comment

Choose a reason for hiding this comment

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

Thanks @SteveDala nice cleanup and useful improvement to the gitaction and dev container

@BrianDangDev BrianDangDev merged commit b095248 into main Feb 2, 2026
6 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.

3 participants