Skip to content

Conversation

@olexale
Copy link
Owner

@olexale olexale commented Oct 6, 2025

This PR adds Gherkin comments support

Fix #115

@olexale olexale requested a review from Copilot October 6, 2025 14:44
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 adds Gherkin comments support to the codebase by implementing parsing and filtering for lines that start with '#'. The implementation ensures that comment lines are ignored during feature file processing.

  • Adds support for recognizing Gherkin comment lines starting with '#'
  • Filters out comment lines during feature file processing
  • Adds comprehensive test coverage for comment handling

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

File Description
lib/src/bdd_line.dart Adds comment line type enum and parsing logic with comment markers
lib/src/feature_file.dart Filters out comment lines during preprocessing
test/comment_test.dart Adds dedicated test case for comment handling
test/full_set_test.dart Updates existing test with comment examples
Comments suppressed due to low confidence (3)

test/comment_test.dart:1

  • Corrected escaping of single quotes in string literal.
import 'package:bdd_widget_test/src/feature_file.dart';

test/comment_test.dart:1

  • Corrected escaping of single quotes in string literal.
import 'package:bdd_widget_test/src/feature_file.dart';

test/comment_test.dart:1

  • Corrected escaping of single quotes in string literal.
import 'package:bdd_widget_test/src/feature_file.dart';

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

@olexale olexale merged commit 2d445e8 into master Oct 6, 2025
1 check passed
@olexale olexale deleted the feature/gherkin-comments branch October 6, 2025 14:53
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.

Incorrect example: top-level Gherkin comments break build_runner

1 participant