Skip to content

gpnf-fillablepdfs-auto-attach-child-pdfs.php: Added "(FillablePDFs)" to the snippet name.#1256

Merged
saifsultanc merged 1 commit intomasterfrom
wizlizard-patch-2
Mar 28, 2026
Merged

gpnf-fillablepdfs-auto-attach-child-pdfs.php: Added "(FillablePDFs)" to the snippet name.#1256
saifsultanc merged 1 commit intomasterfrom
wizlizard-patch-2

Conversation

@wizlizard
Copy link
Copy Markdown
Contributor

Context

At the moment, the snippets for Gravity PDF and FillablePDFs have the same titles.
CleanShot 2026-03-13 at 18 31 37

Summary

Updated the comment header to specify "FillablePDFs" in the HSL search.

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Mar 13, 2026

Walkthrough

A header comment in a plugin file was updated to reflect "FillablePDFs (FillablePDFs)" instead of the previous text. No functional code changes were made.

Changes

Cohort / File(s) Summary
Documentation
gp-nested-forms/gpnf-fillablepdfs-auto-attach-child-pdfs.php
Header comment text updated to clarify the FillablePDFs plugin reference.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description check ✅ Passed The description includes Context and Summary sections as required by the template, providing clear explanation of the issue and the solution implemented.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Title check ✅ Passed The title directly and specifically describes the main change: adding '(FillablePDFs)' to the snippet name in the file header comment to disambiguate it from the Gravity PDF version.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch wizlizard-patch-2

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions
Copy link
Copy Markdown

Fails
🚫

Pull request title does match the correct format. The Pull Request title should match our Snippet Library Pull Request Title Guidelines in Notion.

🚫 Pull request title needs to end in a period or exclamation.
🚫

Commit message 'Added "(FillablePDFs)" to snippet name' does match the correct format. See our Snippet Library Commit Messages Guidelines in Notion.

🚫 Commit message 'Added "(FillablePDFs)" to snippet name' needs to end in a period or exclamation.
Warnings
⚠️ When ready, don't forget to request reviews on this pull request from your fellow wizards.

Generated by 🚫 dangerJS against 213ee1c

@saifsultanc saifsultanc changed the title Added "(FillablePDFs)" to snippet name gpnf-fillablepdfs-auto-attach-child-pdfs.php: Added "(FillablePDFs)" to the snippet name. Mar 28, 2026
@saifsultanc saifsultanc merged commit 20aac29 into master Mar 28, 2026
3 of 5 checks passed
@saifsultanc saifsultanc deleted the wizlizard-patch-2 branch March 28, 2026 13:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants