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.

Sign up for GitHub

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

Jump to bottom

Publish eslint config #1762

Open
grod220 opened this issue Sep 3, 2024 · 2 comments · May be fixed by #1884
Open

Publish eslint config #1762

grod220 opened this issue Sep 3, 2024 · 2 comments · May be fixed by #1884
Assignees
@vacekj
Labels
deployment

Comments

@grod220
Copy link
Collaborator

grod220 commented Sep 3, 2024

A number of penumbra-related projects are interested in consuming the beefy eslint config that has been created here.

TASK:

  • Publish eslint config to npm in a way consumers can easily re-use & customize (override ability)
@github-project-automation github-project-automation bot moved this to 🗄️ Backlog in Penumbra web Sep 3, 2024
@grod220 grod220 mentioned this issue Sep 3, 2024
Open
@grod220 grod220 added this to Labs web Sep 3, 2024
@grod220 grod220 moved this to 📝 Todo in Labs web Sep 3, 2024
@vacekj
Copy link
Member

vacekj commented Sep 9, 2024

I lack publish to npm capabilities, but I published to Github, installation is very easy - readme here: https://github.com/prax-wallet/configs

@vacekj vacekj moved this from 🏗 In progress to 🔎 In review in Labs web Sep 9, 2024
@vacekj
Copy link
Member

vacekj commented Sep 9, 2024

@JasonMHasperhoven tested both the eslint and prettier configs in dex-explorer and found them to work. I am testing rn with the zone-web repo.

@vacekj vacekj self-assigned this Sep 9, 2024
This was referenced Oct 30, 2024
Open
Closed
@grod220 grod220 moved this from 🛑 Blocked to 🏗 In progress in Labs web Nov 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@vacekj vacekj

Labels
deployment
Projects
Penumbra web
Status: 🗄️ Backlog
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat: shared eslint and prettier config penumbra-zone/web
2 participants
@vacekj @grod220