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

Added git hooks and lavamoat package #2

Merged
merged 1 commit into from
Jun 25, 2024
Merged

Conversation

ukorvl
Copy link
Member

@ukorvl ukorvl commented Jun 25, 2024

This diff changes nothing in the source code.
It brings useful editor settings to vscode, adds git hooks to run lint before every commit, fixes version of biomejs schema and adds lavamoat package along with restricting npm scripts by default to prevent supply chain attacks.
More about this package and why it is important when we work with blockchain-related code - https://github.com/LavaMoat/LavaMoat

@ukorvl ukorvl self-assigned this Jun 25, 2024
@ukorvl ukorvl requested a review from 0xAleksaOpacic June 25, 2024 09:29
@ukorvl ukorvl force-pushed the minor-dx-imporvements branch 2 times, most recently from 8672c7b to b84438d Compare June 25, 2024 09:31
@ukorvl ukorvl force-pushed the minor-dx-imporvements branch from b84438d to 294fad5 Compare June 25, 2024 09:40
Copy link
Collaborator

@0xAleksaOpacic 0xAleksaOpacic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for adding this. Appreciate it

@ukorvl ukorvl merged commit 9b92cf7 into master Jun 25, 2024
3 checks passed
@0xAleksaOpacic 0xAleksaOpacic deleted the minor-dx-imporvements branch September 3, 2024 17:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants