Skip to content

Conversation

@clue
Copy link
Owner

@clue clue commented Sep 8, 2025

Builds on top of clue/framework-x#283, #42, #41 and #40

@clue clue requested a review from Copilot September 8, 2025 12:42
Copy link

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

This PR updates integration tests to replace PCRE (Perl Compatible Regular Expressions) with extended grep patterns for better compatibility and portability.

  • Replaces -P (PCRE) flags with -E (extended regex) in grep commands
  • Adds skipifnot() function for conditional test skipping
  • Updates regex patterns to work with extended grep syntax
  • Modifies curl output handling to strip carriage returns

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@clue clue merged commit 06b68f6 into clue:main Sep 8, 2025
2 checks passed
@clue clue deleted the egrep branch September 8, 2025 12:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant