-
-
Notifications
You must be signed in to change notification settings - Fork 47
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Doc update #172
Doc update #172
Conversation
README.md
Outdated
- [Objective-C](./objective-c) - Not tested? | ||
- [Perl](./perl) - Tested on Perl 5.14 and 5.36 | ||
- [PHP](./php) - Tested on PHP 8.1 | ||
- [Dart](./dart) - Tested on Dart 1.4 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is perishable information. It will go out of date when the Github Action is changed.
Consider removing the versions altogether and a second that describes how Gherkin is tested, refering to the CI files for specific versions.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fair point. But can you help with the 2 misunderstood ones? C/Obj C? Those appear wrong / untested
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
C is tested by
https://github.com/cucumber/gherkin/blob/main/.github/workflows/test-c.yml
Objective-C is untested.
Maybe put that in the readme for objective-c.
🤔 What's changed?
Add testing notes RE versions in readme
⚡️ What's your motivation?
Get more clarity on what is tested
🏷️ What kind of change is this?
♻️ Anything particular you want feedback on?
📋 Checklist:
This text was originally generated from a template, then edited by hand. You can modify the template here.