Skip to content

Adds a basic example#17

Merged
donatj merged 1 commit into
masterfrom
example
Jun 3, 2026
Merged

Adds a basic example#17
donatj merged 1 commit into
masterfrom
example

Conversation

@donatj

@donatj donatj commented Jun 3, 2026

Copy link
Copy Markdown
Member

No description provided.

@donatj donatj marked this pull request as ready for review June 3, 2026 15:38
Copilot AI review requested due to automatic review settings June 3, 2026 15:38

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Adds an end-to-end usage example for Quorum\DotNotation\DotNotationParser and wires it into the generated README workflow so users can see both the code and expected output.

Changes:

  • Added an “Example” section to the README including sample code and its output.
  • Added a runnable example script under example/example.php.
  • Updated .mddoc.xml.dist to include the example source and execute it when generating README.md.
  • Added a PHP-focused .editorconfig to standardize basic editor behavior (tabs, LF, final newline).

Reviewed changes

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

File Description
README.md Adds a new Example section with sample code + expected output.
example/example.php Introduces a runnable script demonstrating parsing scenarios.
.mddoc.xml.dist Includes the example source and command output in README generation.
.editorconfig Defines baseline PHP editor settings (tabs, LF, final newline).

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

Comment thread README.md
Comment thread example/example.php
@donatj donatj merged commit d16fe08 into master Jun 3, 2026
15 checks passed
@donatj donatj deleted the example branch June 3, 2026 16:15
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