Skip to content
This repository was archived by the owner on Oct 6, 2025. It is now read-only.

Comments

Notifications in qt app#75

Merged
mjorgecruz merged 10 commits intodevfrom
notifications-in-qt-app
May 15, 2025
Merged

Notifications in qt app#75
mjorgecruz merged 10 commits intodevfrom
notifications-in-qt-app

Conversation

@ziliolu
Copy link
Contributor

@ziliolu ziliolu commented May 15, 2025

This pull request includes significant updates to issue templates, the pull request template, and the CMakeLists.txt file. The changes aim to streamline the issue reporting process, improve the pull request documentation workflow, and update build configurations for new components.

Issue Template Updates:

  • Removed YAML-based templates for bug reports (bug-report.yml) and feature requests (feature-request.yml) and replaced them with Markdown-based templates for external contributors (external-contribution-bug-report.md and external-contribution-feature-request.md). These new templates simplify the structure and improve clarity. [1] [2] [3] [4]
  • Removed references to related epics, stories, and tasks in epic-template.yml, story-template.yml, and task-template.yml to simplify issue creation. [1] [2] [3]
  • Deleted the config.yml file, which previously linked YAML templates to GitHub's issue template chooser.

Pull Request Template Update:

  • Replaced the detailed pull request template with a minimal placeholder, encouraging contributors to use Copilot-generated overviews and add any additional context as needed. This reduces overhead and focuses on automation.

Build Configuration Updates:

  • Updated CMakeLists.txt to add support for new components (HandClusterB and MiddleWareApp) while deprecating the InstrumentClusterApp. Adjustments include new resource files, include directories, and executable targets.

Copy link
Contributor

@mjorgecruz mjorgecruz left a comment

Choose a reason for hiding this comment

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

All ok. Great work.

@mjorgecruz mjorgecruz merged commit fc725b1 into dev May 15, 2025
1 check failed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants