Skip to content

Commit

Permalink
Update contribution guide, bug repot and pre-commit. (#71)
Browse files Browse the repository at this point in the history
  • Loading branch information
antejavor authored Nov 7, 2024
1 parent 33aa388 commit 0faf34a
Show file tree
Hide file tree
Showing 4 changed files with 23 additions and 29 deletions.
21 changes: 21 additions & 0 deletions .github/ISSUE_TEMPLATE/bug_report.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,21 @@
---
name: Bug report
about: Create a report to help us improve
title: ""
labels: bug
---

**Chart type**
Specify the chart type you experienced issues with (Memgraph Standalone, Memgraph HA, Memgraph LAB)

**Chart version:**
Specify the version of the chart you are running.

**What happened?**
Describe the issue and what you expected to happen.

**Environment:**
Specify the environment where the issue occurred and how it could be related to the bug.

**Relevant log output:**
Please copy and paste any relevant log output from pods and containers.
27 changes: 0 additions & 27 deletions .github/ISSUE_TEMPLATE/bug_report.yml

This file was deleted.

2 changes: 1 addition & 1 deletion .pre-commit-config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -4,9 +4,9 @@ repos:
hooks:
- id: trailing-whitespace
- id: end-of-file-fixer
- id: check-json
- id: check-yaml
exclude: ^charts/(memgraph|memgraph-lab|memgraph-high-availability)/templates/
- id: check-json
- id: mixed-line-ending
- id: check-merge-conflict
- id: detect-private-key
2 changes: 1 addition & 1 deletion CONTRIBUTING.md
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,7 @@ There are two main types of contributions:
1. **Bug fixes**
2. **New features**

Both types of fixes should be applied to the `release` branch, that is currently active based on the upcoming [milestone](https://github.com/memgraph/helm-charts/milestone/2) for example `release-0.1.3` branch.
Both types of fixes should be applied to the `main` branch.

In order for a pull request to be merged, a review by two code owners is required and the tests need to pass remotely.

Expand Down

0 comments on commit 0faf34a

Please sign in to comment.