Skip to content

fix: add steps and step-definitions to SPEC_FILE_PATTERN regex#200

Merged
mccmrunal merged 2 commits into
mainfrom
fix/cucumber-steps-regex-pattern
May 11, 2026
Merged

fix: add steps and step-definitions to SPEC_FILE_PATTERN regex#200
mccmrunal merged 2 commits into
mainfrom
fix/cucumber-steps-regex-pattern

Conversation

@mccmrunal
Copy link
Copy Markdown
Contributor

Updated to accurately describe the export as a public, evolving pattern rather than a legacy stub

Copilot AI review requested due to automatic review settings May 11, 2026 13:46
@mccmrunal mccmrunal force-pushed the fix/cucumber-steps-regex-pattern branch from 8f2fff0 to 28496fa Compare May 11, 2026 13:48
@mccmrunal mccmrunal merged commit f8e81ad into main May 11, 2026
5 checks passed
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the exported SPEC_FILE_PATTERN used by the service to recognize test-related paths, and clarifies its intended role as a shared public pattern for consumers.

Changes:

  • Updated SPEC_FILE_PATTERN to match step_definitions via step[-_]?definitions.
  • Reworded the JSDoc for SPEC_FILE_PATTERN to describe it as a public, evolving shared pattern.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines +101 to +105
* Shared pattern for identifying test-related file paths.
* Kept as a public export for consumers and may be expanded as supported locations evolve.
*/
export const SPEC_FILE_PATTERN =
/\/(test|spec|features|steps|step-definitions|pageobjects|@wdio\/expect-webdriverio)\//i
/\/(test|spec|features|steps|step[-_]?definitions|pageobjects|@wdio\/expect-webdriverio)\//i
export const SPEC_FILE_PATTERN =
/\/(test|spec|features|steps|step-definitions|pageobjects|@wdio\/expect-webdriverio)\//i
/\/(test|spec|features|steps|step[-_]?definitions|pageobjects|@wdio\/expect-webdriverio)\//i

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.

2 participants