Skip to content
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

Each example should have its own endpoints.yaml file #50

Open
php-coder opened this issue Apr 14, 2024 · 0 comments
Open

Each example should have its own endpoints.yaml file #50

php-coder opened this issue Apr 14, 2024 · 0 comments
Assignees
Labels
decision Requires making a decision
Milestone

Comments

@php-coder
Copy link
Owner

php-coder commented Apr 14, 2024

Pros:

  1. endpoints.yaml for language without types won't contain additional information about fields types
  2. allows us to add a new feature gradually (by updating relevant endpoints.yaml instead of a global one)
  3. later it will help us to detect changes only for particular language and skip other tests
  4. it became possible to use different SQL queries. Especially useful for SQLite (Support SQLite #53) that doesn't have NOW() function
    Cons:
  5. duplication
  6. the integration tests also should be split
@php-coder php-coder added the decision Requires making a decision label Apr 14, 2024
@php-coder php-coder added this to the 0.0.3 milestone Apr 14, 2024
@php-coder php-coder self-assigned this Apr 14, 2024
@php-coder php-coder modified the milestones: 0.0.3, 0.0.4 Apr 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
decision Requires making a decision
Projects
None yet
Development

No branches or pull requests

1 participant