Skip to content
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

Add security announcements to the release notes #1158

Merged
merged 1 commit into from
Oct 15, 2024

Conversation

mickhawkins
Copy link
Collaborator

The AI linting spelling error due to a PHP filename is causing linting failures on a bunch of other pull requests, so fixing it here on the commit before my release notes additions.

Copy link

netlify bot commented Oct 15, 2024

Deploy Preview for moodledevdocs ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit f8faa8d
🔍 Latest deploy log https://app.netlify.com/sites/moodledevdocs/deploys/670e1a18b5687f0007d3d6e1
😎 Deploy Preview https://deploy-preview-1158--moodledevdocs.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 site configuration.

Copy link
Contributor

github-actions bot commented Oct 15, 2024

⚡️ Lighthouse report for the deploy preview of this PR

URL Performance Accessibility Best Practices SEO PWA Report
/ 🟠 69 🟢 95 🟢 100 🟢 90 🟢 100 Report
/docs/4.4/apis/commonfiles 🟠 57 🟢 93 🟢 100 🟢 100 🟢 100 Report
/general/development/gettingstarted 🟠 65 🟢 95 🟢 100 🟢 90 🟢 100 Report
/general/releases 🟠 60 🟢 95 🟢 100 🟢 100 🟢 100 Report

@andrewnicols
Copy link
Member

This is wrong. There are four backticks together in another part of the file. See #1151

@mickhawkins mickhawkins changed the title Fix linting error in AI docs, and add security announcements to the release notes Add security announcements to the release notes Oct 15, 2024
@mickhawkins
Copy link
Collaborator Author

Thanks @andrewnicols, I've removed the linting fix commit.

@HuongNV13
Copy link
Member

Thanks, Mick.
Looks good to me

@HuongNV13 HuongNV13 added this pull request to the merge queue Oct 15, 2024
Merged via the queue into moodle:main with commit 9f8458c Oct 15, 2024
7 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