Address review (ilidemi)

Jeremyyang920 · claude · Jeremyyang920 · commit 1287d2ab18ac · 2026-06-24T15:58:21.000-07:00
- labeler.yml: header comment lists S3 as deprecated and notes ClickHouse/Postgres as the excluded (maintained) destinations; add the BigQuery destination QRep file (qrep.go) to the tracked paths. - CONTRIBUTING.md: add S3 to the deprecated list; note BigQuery is deprecated only as a destination. - bug_report.yml: drop the duplicate migration-guide link. - feature_request.yml: new issue-form template so feature requests have a home while the chooser stays curated (blank issues remain disabled). - Issue forms: add an optional "Source connector" dropdown and make the destination dropdown optional, so source-side issues aren't forced to pick a destination (GitHub issue forms have no conditional fields). The destination dropdown keeps its exact label so the deprecation-triage workflow still parses it. Refs #4465 Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
diff --git a/.github/ISSUE_TEMPLATE/bug_report.yml b/.github/ISSUE_TEMPLATE/bug_report.yml
@@ -38,20 +38,34 @@ body:
     validations:
       required: true
 
+  - type: dropdown
+    id: source_connector
+    attributes:
+      label: Source connector
+      description: If this issue relates to a source connector, select it here. Leave blank if not applicable.
+      options:
+        - Postgres
+        - MySQL
+        - MongoDB
+        - BigQuery
+        - Other
+    validations:
+      required: false
+
   - type: markdown
     attributes:
       value: |
         ### ⚠️ Deprecated destination connectors
 
         The following destination connectors are **deprecated**: **Snowflake**, **BigQuery**, **ElasticSearch**, **Kafka**, **Redpanda**, **Confluent**, **Azure Event Hubs**, **Google Pub/Sub**, and **S3**.
 
-        These destinations are no longer actively maintained, but remain functional. Bugs affecting only these destinations are unlikely to be prioritized. Note: BigQuery is deprecated **only as a destination** — it remains a supported source. See the [deprecated connectors documentation](https://github.com/PeerDB-io/peerdb/blob/main/docs/deprecated-connectors.md) ([../../docs/deprecated-connectors.md](../../docs/deprecated-connectors.md)) for details and migration guidance.
+        These destinations are no longer actively maintained, but remain functional. Bugs affecting only these destinations are unlikely to be prioritized. Note: BigQuery is deprecated **only as a destination** — it remains a supported source. See the [deprecated connectors documentation](https://github.com/PeerDB-io/peerdb/blob/main/docs/deprecated-connectors.md) for details and migration guidance.
 
   - type: dropdown
     id: destination_connector
     attributes:
       label: Destination connector
-      description: Which destination connector is affected? (See the deprecation notice above.)
+      description: If this issue relates to a destination connector, select it here (see the deprecation notice above). Leave blank if not applicable.
       options:
         - ClickHouse
         - ClickHouse Cloud
@@ -65,6 +79,6 @@ body:
         - Confluent
         - Azure Event Hubs
         - Google Pub/Sub
-        - Other / N/A
+        - Other
     validations:
-      required: true
+      required: false
diff --git a/.github/ISSUE_TEMPLATE/feature_request.yml b/.github/ISSUE_TEMPLATE/feature_request.yml
@@ -0,0 +1,77 @@
+name: Feature request
+description: Suggest a feature or improvement for PeerDB
+title: "[Feature]: "
+labels: ["enhancement"]
+body:
+  - type: markdown
+    attributes:
+      value: |
+        Thanks for suggesting an improvement! Please describe the problem you are trying to solve and what you would like to happen.
+
+  - type: textarea
+    id: problem
+    attributes:
+      label: Problem / motivation
+      description: What problem are you trying to solve, and what is the use case?
+      placeholder: When replicating ..., I need ... but currently ...
+    validations:
+      required: true
+
+  - type: textarea
+    id: proposal
+    attributes:
+      label: Proposed solution
+      description: Describe what you would like to happen.
+    validations:
+      required: true
+
+  - type: textarea
+    id: alternatives
+    attributes:
+      label: Alternatives considered
+      description: Any alternative solutions or workarounds you have considered.
+    validations:
+      required: false
+
+  - type: dropdown
+    id: source_connector
+    attributes:
+      label: Source connector
+      description: If this request relates to a source connector, select it here. Leave blank if not applicable.
+      options:
+        - Postgres
+        - MySQL
+        - MongoDB
+        - BigQuery
+        - Other
+    validations:
+      required: false
+
+  - type: markdown
+    attributes:
+      value: |
+        ### ⚠️ Deprecated destination connectors
+
+        The following destination connectors are **deprecated** and no longer actively maintained: **Snowflake**, **BigQuery**, **ElasticSearch**, **Kafka**, **Redpanda**, **Confluent**, **Azure Event Hubs**, **Google Pub/Sub**, and **S3**. New features for these destinations are unlikely to be prioritized. Note: BigQuery is deprecated **only as a destination** — it remains a supported source. See the [deprecated connectors documentation](https://github.com/PeerDB-io/peerdb/blob/main/docs/deprecated-connectors.md).
+
+  - type: dropdown
+    id: destination_connector
+    attributes:
+      label: Destination connector
+      description: If this request relates to a destination connector, select it here (see the deprecation notice above). Leave blank if not applicable.
+      options:
+        - ClickHouse
+        - ClickHouse Cloud
+        - Postgres
+        - S3
+        - Snowflake
+        - BigQuery
+        - ElasticSearch
+        - Kafka
+        - Redpanda
+        - Confluent
+        - Azure Event Hubs
+        - Google Pub/Sub
+        - Other
+    validations:
+      required: false
diff --git a/.github/labeler.yml b/.github/labeler.yml
@@ -1,7 +1,7 @@
 # Configuration for actions/labeler@v5.
 # Applies the `deprecated` label to PRs that touch deprecated destination connectors:
-# Snowflake, BigQuery, ElasticSearch, Kafka (incl. Confluent/Redpanda), Azure Event Hubs, Google Pub/Sub.
-# S3 and ClickHouse are intentionally excluded.
+# Snowflake, BigQuery, ElasticSearch, Kafka (incl. Confluent/Redpanda), Azure Event Hubs, Google Pub/Sub, S3.
+# ClickHouse and Postgres (the actively-maintained destinations) are intentionally excluded.
 deprecated:
   - changed-files:
       - any-glob-to-any-file:
@@ -44,6 +44,7 @@ deprecated:
           # silently disables the whole labeler config in actions/labeler v5.)
           - flow/connectors/bigquery/bigquery.go
           - flow/connectors/bigquery/merge_stmt_generator.go
+          - flow/connectors/bigquery/qrep.go
           - flow/connectors/bigquery/qrep_avro_sync.go
           - flow/connectors/bigquery/clustering.go
           - flow/e2e/bigquery_test.go
diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md
@@ -4,6 +4,6 @@ Thanks for your interest in contributing to PeerDB! Bug reports, feature request
 
 ## Deprecated connectors
 
-Several destination connectors (Snowflake, BigQuery, ElasticSearch, Kafka including Confluent and Redpanda, Azure Event Hubs, and Google Pub/Sub) are deprecated and no longer actively maintained. They remain fully functional, and no code is being removed.
+Several destination connectors (Snowflake, BigQuery, ElasticSearch, Kafka including Confluent and Redpanda, Azure Event Hubs, Google Pub/Sub, and S3) are deprecated and no longer actively maintained. They remain fully functional, and no code is being removed. (BigQuery is deprecated only as a destination — it remains a supported source.)
 
 If you depend on one of these connectors, see the [deprecated connectors migration guide](docs/deprecated-connectors.md) for how to pin to a release or fork the relevant connector code.
