From fd605a2852368a91b3fcee6dda1f824e6db886c1 Mon Sep 17 00:00:00 2001
From: mringler <m@mringler.net>
Date: Sat, 21 Mar 2026 12:01:06 +0100
Subject: [PATCH 1/2] Add PR template

---
 .github/pull_request_template.md | 58 ++++++++++++++++++++++++++++++++
 1 file changed, 58 insertions(+)
 create mode 100644 .github/pull_request_template.md

diff --git a/.github/pull_request_template.md b/.github/pull_request_template.md
new file mode 100644
index 0000000000..8498beb05a
--- /dev/null
+++ b/.github/pull_request_template.md
@@ -0,0 +1,58 @@
+## Issue
+<!--
+Please add a short description of the current behavior (without your changes),
+i.e. "
+    When setting the `phpType` column attribute to a PHP Enum, Perpl models try to instantiate the value
+    with `new`. This leads to exceptions, since Enums have to be referenced as class constants or via `::from()`.
+"-->
+
+
+
+## Changes
+<!--
+Please add a short description of your changes,
+i.e. "
+    This PR checks if `phpType` is an enum and instantiates those correctly. It affects loading from DB (hydrate()),
+    writing to DB (getAccessValueStatement()) and applying default values. 
+"-->
+
+
+
+## Implementation Details
+<!--
+Please add anything that helps us to reason about the changes (optional),
+i.e. "
+    - I'm handling BackedEnum and UnitEnum as different cases, because of how different they are. Does that make sense?
+    - Visibility of `FooClass::fooMethod()` was changed to public, so I can call it from `SomeOtherClass`.
+"-->
+
+
+
+## Test strategy
+<!--
+Your changes will have to pass static analysis, linter, and the test suite, and any added or changed behavior needs to be validated by tests, provided by your PR.
+
+Please add a short description of the tests required to validate this PR,
+i.e. "
+    - Added tests to check code output as string.
+    - Runtime behavior is validated by creating dummy classes with QuickBuilder.
+
+    Not sure if this requires actual runtime tests against database?
+"
+
+ Find out more about test suite at https://perplorm.github.io/documentation/cookbook/working-with-test-suite.html
+ 
+ You can run checks locally using the docker containers at https://github.com/perplorm/perpl-test-docker
+
+-->
+
+
+
+## Notes
+<!--
+Remove if not needed, or add what you want to discuss.
+
+!!! Thank you for contributing to Perpl !!!
+
+Once submitted, we'll go through a review to get your changes merged as soon as possible.
+-->

From 6ade9f262bb15be6960955babc13aed303a80c73 Mon Sep 17 00:00:00 2001
From: mringler <m@mringler.net>
Date: Sat, 21 Mar 2026 12:27:35 +0100
Subject: [PATCH 2/2] Omit CI when just .md files change, check concurrency

---
 .github/workflows/ci.yml | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
index 1211107b08..669115e496 100644
--- a/.github/workflows/ci.yml
+++ b/.github/workflows/ci.yml
@@ -6,8 +6,14 @@ on:
       - 'main'
       - 'develop'
   pull_request:
+    paths-ignore:
+      - '**.md'
   workflow_dispatch:
 
+concurrency:
+  group: ${{ github.workflow }}-${{ github.ref }}
+  cancel-in-progress: true
+
 env:
   DB_NAME: 'perpl_tests'
   DB_USER: 'perpl'