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

Siz #219

Merged
merged 12 commits into from
Feb 20, 2024
Merged

Siz #219

merged 12 commits into from
Feb 20, 2024

Conversation

InnopolisUni
Copy link
Owner

Pull Request Template for InnoFW

Title: [Short description of your changes]

Description

Please provide a detailed description of your changes, including:

  1. What issue(s) are you addressing?
  2. How have you tested your changes?
  3. If applicable, any new dependencies added and why they are necessary.

Pre-commit Hook Activation and Linter Checks

Please ensure you have activated the pre-commit hook to enable automatic linting checks before submitting your PR. This helps maintain code quality and consistency throughout the project.

Documentation

Please ensure you have documented the functionality of your changes, especially if it introduces new features or affects existing ones. Proper documentation ensures that the project remains maintainable and understandable for other contributors.

Changes

  • List major changes made in bullet points
  • Include any minor changes or refactoring

What type of PR is this? (check all applicable)

  • Refactor
  • Feature
  • Bug Fix
  • Optimization
  • Documentation Update

Checklist

  • I have tested my changes locally and all tests passed
  • I have updated the documentation (if necessary)
  • I have added new dependencies (if necessary)
  • My changes do not introduce any breaking changes or conflicts with existing code
  • I have activated the pre-commit hook and ensured that linting checks pass

Added/updated tests?

  • Yes
  • No, and this is why: please replace this line with details on why tests
    have not been included
  • I need help with writing tests

Additional Notes

Provide any additional information or context that may be helpful for the reviewer.


Thank you for your contribution to InnoFW! Your efforts help make this project better for everyone.

BarzaH
BarzaH previously approved these changes Feb 20, 2024
@BarzaH BarzaH removed the request for review from KGallyamov February 20, 2024 11:10
BarzaH
BarzaH previously approved these changes Feb 20, 2024
@BarzaH BarzaH merged commit 2ae3df4 into main Feb 20, 2024
1 check passed
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