Skip to content

MDL-87988 [docs] Add the TypeScript coding style - #1726

Merged
abgreeve merged 1 commit into
moodle:mainfrom
meirzamoodle:MDL-87988_docs
Nov 3, 2026
Merged

abgreeve merged 1 commit into
moodle:mainfrom
meirzamoodle:MDL-87988_docs

Conversation

@meirzamoodle

Copy link
Copy Markdown
Collaborator

TypeScript is now linted with XO. The new page lists the plugins and Moodle's overrides, and the build tools guide no longer points to the removed eslint:react task.

@netlify

netlify Bot commented Oct 14, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for moodledevdocs ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit 4673a47
🔍 Latest deploy log https://app.netlify.com/projects/moodledevdocs/deploys/6ae846e14df3675f2f42395b
😎 Deploy Preview https://deploy-preview-1726--moodledevdocs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

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

@meirzamoodle meirzamoodle added the blocked-by-tracker This issue is blocked by an issue in https://tracker.moodle.org/ label Oct 30, 2026
TypeScript is now linted with XO. The new page lists the plugins and Moodle's overrides, and the build tools guide no longer points to the removed eslint:react task.

@abgreeve abgreeve left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks Meirza for creating this documentation for the TypeScript linting.

@abgreeve
abgreeve added this pull request to the merge queue Nov 3, 2026
Merged via the queue into moodle:main with commit 5c3ab11 Nov 3, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

blocked-by-tracker This issue is blocked by an issue in https://tracker.moodle.org/

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants