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

Create eslint-plugin-hpe-design-system #30

Closed
5 tasks done
taysea opened this issue Dec 20, 2021 · 5 comments
Closed
5 tasks done

Create eslint-plugin-hpe-design-system #30

taysea opened this issue Dec 20, 2021 · 5 comments

Comments

@taysea
Copy link
Collaborator

taysea commented Dec 20, 2021

Add config rules that are specific to HPE Design System. Starting an issue to create a running list of rules:

@halocline
Copy link
Collaborator

halocline commented Dec 21, 2021

Discussion points:

  • Do HPE Design System rules belong on eslint-plugin-grommet? Or should eslint-plugin-grommet only contain rules related to general Grommet and accessibility best practices?
  • When should a rule be a warning vs. error? Determined that rules should be warnings
  • @taysea any other discussion points?

@ericsoderberghp
Copy link
Collaborator

Good questions!
If we expect to have more than just a couple of rules, I'm wondering if we should have eslint-plugin-hpe-design-system?

@taysea taysea changed the title Enhance eslint-plugin-grommet with HPE Design System config Create eslint-plugin-hpe-design-system Jun 2, 2023
@taysea
Copy link
Collaborator Author

taysea commented Jun 2, 2023

Right now I'm leaning towards creating eslint-plugin-hpe-design-system. It should extend the "recommended" config from eslint-plugin-grommet but exist as its own repo since rules will be more specific to HPE, as opposed to open source community.

Aligns with mentality to keep Grommet "open-source" focused.

@taysea
Copy link
Collaborator Author

taysea commented Jul 20, 2023

eslint-plugin-hpe-design-system has been created: https://github.com/grommet/eslint-plugin-hpe-design-system/tree/main

Before closing this, I want to discuss the remaining checkbox items and if they're relevant more to grommet or HPE Design System

@taysea
Copy link
Collaborator Author

taysea commented Jul 24, 2023

Closing this, issues have been created.

@taysea taysea closed this as completed Jul 24, 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

3 participants