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

From develop #143

Closed
wants to merge 9 commits into from
Closed

From develop #143

wants to merge 9 commits into from

Conversation

verytactical
Copy link
Collaborator

We ended up with 2 dev branches.

These are fixes from develop, merged into staging.

Ilya Egorov and others added 7 commits October 22, 2024 22:24
With a new `develop` git workflow it turned out Actions don't run on PRs to branches except for
`main`.
npm audit complains about several vulnerabilities. This patch is the result of `npm audit fix`. Fix
for `next` requires major version update, isn't critical, and we plan to get rid of it anyway, so it
wasn't updated.
"Prepare commit message" hook runs not only on commit, but also on a variety of other git tasks such
as rebase and merge, sometimes in an automated way from IDE (Jetbrains IDEs do that). git-cz utility
wasn't built with this in mind, and leads to bad developer experience. We can use already existing
`npm run commit` script for this. Also `"prepare"` npm hook runs when a package is packaged (`npm
pack`), and `husky` is intended to be run on install.
@verytactical verytactical requested a review from Mobyman as a code owner October 29, 2024 12:35
@anton-trunov
Copy link
Member

what is the status of this PR?

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.

3 participants