Skip to content

Add: script for fixing linter issues#1440

Merged
Sakoram merged 2 commits intomainfrom
gh-copilot-memory
Feb 18, 2026
Merged

Add: script for fixing linter issues#1440
Sakoram merged 2 commits intomainfrom
gh-copilot-memory

Conversation

@Sakoram
Copy link
Collaborator

@Sakoram Sakoram commented Feb 17, 2026

Doc: Improve copilot instructions

Create more detailed documentation for copilot instructions, including architecture and design philosophy, concurrency and locking, DPDK usage patterns, memory management, pacing/timing/performance, session lifecycle/dataflow, and threading/scheduler.

@Sakoram Sakoram force-pushed the gh-copilot-memory branch 2 times, most recently from f98ce93 to e67b014 Compare February 17, 2026 16:23
@Sakoram Sakoram changed the title Doc: Improve copilot instructions Add: script for fixing markdown linting issues Feb 17, 2026
@Sakoram Sakoram added TO_REBASE Please don't squash this PR, use the rebase option during merging process labels Feb 17, 2026
Copy link
Contributor

@soopel soopel left a comment

Choose a reason for hiding this comment

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

Great stuff!

@Sakoram Sakoram changed the title Add: script for fixing markdown linting issues Add: script for fixing linter issues Feb 17, 2026
Create more detailed documentation for copilot instructions,
including architecture and design philosophy, concurrency and locking,
DPDK usage patterns, memory management, pacing/timing/performance,
session lifecycle/dataflow, and threading/scheduler.

Signed-off-by: Kasiewicz, Marek <marek.kasiewicz@intel.com>
Copy link
Collaborator

@DawidWesierski4 DawidWesierski4 left a comment

Choose a reason for hiding this comment

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

please change the name of the script ot the

fix_lint.sh

tbh i would just add it to format coding all of this here

whats the point of separating those 2
Also i would just add the settings in some yml and be done with it, why create some temporary yml ?

i don't get the benefit?

Please and thank you

Copy link
Collaborator

@DawidWesierski4 DawidWesierski4 left a comment

Choose a reason for hiding this comment

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

LGTM 🥈

@Sakoram Sakoram force-pushed the gh-copilot-memory branch 3 times, most recently from f900578 to f9b0b92 Compare February 18, 2026 09:55
@Sakoram Sakoram enabled auto-merge (rebase) February 18, 2026 10:04
Signed-off-by: Kasiewicz, Marek <marek.kasiewicz@intel.com>
Copy link
Collaborator

@DawidWesierski4 DawidWesierski4 left a comment

Choose a reason for hiding this comment

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

Would be lovly if the format coding clang also had "Check command"
But tbh it is fine as it is

@Sakoram Sakoram merged commit 710f23b into main Feb 18, 2026
26 of 29 checks passed
@Sakoram Sakoram deleted the gh-copilot-memory branch February 18, 2026 11:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

TO_REBASE Please don't squash this PR, use the rebase option during merging process

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

Comments