Skip to content

Conversation

@anastanei
Copy link
Collaborator

Type of PR 🍩

  • Feature
  • Bugfix
  • Hotfix
  • Refactoring
  • Documentation
  • Infrastructure

Changes Introduced 🥯

  • Updated linting setup (ESLint, Stylelint)
  • Added pre-push hook for tests, lint, and type-check
  • Added pull request template

Self-Check ✅

  • No console errors (except API requests)
  • 80%+ test coverage
  • Reviewers have been requested
  • Documentation updated (if relevant)
  • Package.json scripts updated (if relevant)

@@ -0,0 +1,23 @@
# Type of PR 🍩
Copy link
Collaborator

Choose a reason for hiding this comment

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

Definitely like our PR headings😍


- [ ] No console errors (except API requests)
- [ ] 80%+ test coverage
- [ ] Reviewers have been requested
Copy link
Collaborator

Choose a reason for hiding this comment

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

Mey be it would be better to remove this checkbox since we automated this action

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Just to clarify - which checkbox exactly did you mean?

Copy link
Collaborator

Choose a reason for hiding this comment

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

'Reviewers have been requested'-this one

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Thanks! It's updated

Copy link
Collaborator

Choose a reason for hiding this comment

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

A huge amount of work has been done here, nice!

Copy link
Owner

@Sepulator Sepulator left a comment

Choose a reason for hiding this comment

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

My appreciate of detailed eslint rules.

@anastanei anastanei merged commit 3a5c663 into develop Sep 4, 2025
1 check passed
@anastanei anastanei deleted the chore/pr-template-and-linting-improvement branch September 4, 2025 09:20
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.

4 participants