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

Validate which directives can co-exist #98

Open
CNAChino opened this issue Jul 4, 2022 · 2 comments
Open

Validate which directives can co-exist #98

CNAChino opened this issue Jul 4, 2022 · 2 comments
Assignees

Comments

@CNAChino
Copy link
Contributor

CNAChino commented Jul 4, 2022

Is your feature request related to a problem? Please describe.

No, just an enhancement on validation

Describe the solution you'd like

I'm thinking this should be done on pre-merge. A map of directives it is not allowed to co exist. Example:

@resolver -> @external, @requires, @provides, ...

@ashpak-shaikh
Copy link
Contributor

Whenever there is the following pair of directives that exist together, throw error

resolver and external
resolver and provides
resolver and requires
resolver and provides

@vijayakinbox
Copy link

Hi Ashpak, I will work on this issue. Please assign it to me.

@CNAChino CNAChino changed the title Validate which directives can co-exist on a field definition Validate which directives can co-exist Oct 31, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants