Skip to content

Add missing trailing period.#132

Merged
dkotter merged 1 commit intoWordPress:developfrom
aslamdoctor:fix/error-message-missing-trailing-period
Dec 2, 2025
Merged

Add missing trailing period.#132
dkotter merged 1 commit intoWordPress:developfrom
aslamdoctor:fix/error-message-missing-trailing-period

Conversation

@aslamdoctor
Copy link
Contributor

@aslamdoctor aslamdoctor commented Dec 2, 2025 •
edited
Loading

What?

Closes
#127

Add trailing period to error message.

Why?

How?

Testing Instructions

  1. install plugin
  2. activate plugin
  3. go to Settings > AI Experiments and see error message

Testing Instructions for Keyboard

Screenshots or screencast

Before After
image image

@github-actions
Copy link

github-actions bot commented Dec 2, 2025 •
edited by dkotter
Loading

The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the props-bot label.

If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message.

Co-authored-by: jeffpaul <jeffpaul@git.wordpress.org>
Co-authored-by: aslamdoctor <aslamdoctor@git.wordpress.org>
Co-authored-by: mindctrl <mindctrl@git.wordpress.org>
Co-authored-by: dkotter <dkotter@git.wordpress.org>

To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook.

@codecov
Copy link

codecov bot commented Dec 2, 2025 •
edited
Loading

Codecov Report

❌ Patch coverage is 0% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 27.33%. Comparing base (8cc94c8) to head (07e92e1).
⚠️ Report is 17 commits behind head on develop.

Files with missing lines Patch % Lines
includes/Settings/Settings_Page.php 0.00% 2 Missing ⚠️
Additional details and impacted files 
@@            Coverage Diff             @@
##             develop     #132   +/-   ##
==========================================
  Coverage      27.33%   27.33%           
  Complexity       152      152           
==========================================
  Files             14       14           
  Lines            900      900           
==========================================
  Hits             246      246           
  Misses           654      654
Flag Coverage Δ
unit 27.33% <0.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

mindctrl
mindctrl approved these changes Dec 2, 2025
View reviewed changes
Copy link
Contributor

@mindctrl mindctrl left a comment

Choose a reason for hiding this comment

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

I noticed this yesterday. Thanks for the quick fix.

@dkotter dkotter added this to the 0.2.0 milestone Dec 2, 2025
@dkotter dkotter linked an issue Dec 2, 2025 that may be closed by this pull request
Error message missing trailing period #127
Closed
6 tasks
@dkotter dkotter merged commit 0e56479 into WordPress:develop Dec 2, 2025
23 of 27 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@dkotter dkotter dkotter approved these changes

+1 more reviewer

@mindctrl mindctrl mindctrl approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@aslamdoctor aslamdoctor

Labels

None yet

Projects

None yet

Milestone

0.2.0

Development

Successfully merging this pull request may close these issues.

Error message missing trailing period

3 participants

@aslamdoctor @dkotter @mindctrl