Skip to content

feat: added result summary implementation to parsers and analysers.#235

Merged
cvandeplas merged 89 commits into
mainfrom
issue-169-cached-result-summaries
Jun 23, 2026
Merged

feat: added result summary implementation to parsers and analysers.#235
cvandeplas merged 89 commits into
mainfrom
issue-169-cached-result-summaries

Conversation

@dario-br

Copy link
Copy Markdown
Collaborator

Closes #169

@dario-br

dario-br commented Apr 28, 2026

Copy link
Copy Markdown
Collaborator Author

Eventually this PR brings a little bit more than just the summary, but also:

  • migrate from pycodestyle to ruff to avoid code inconsistences. Since ruff is the tooling configured in the pyproject.toml.
  • Due to that migration a lot of ruff fmt changes.

See more detailed explanation below.

cvandeplas and others added 22 commits June 16, 2026 10:20
- Add commit-message-helper.kiro.hook to generate conventional commit messages
- Add development-rules.md steering file for general development guidelines
- Add parser-analyser-guidelines.md steering file for parser/analyser development
- Enable all fixable rules in ruff configuration for automated code fixes
- Improves code quality, consistency, and developer workflow automation

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 157 out of 157 changed files in this pull request and generated 4 comments.

Comment thread src/sysdiagnose/parsers/plists.py
Comment thread src/sysdiagnose/utils/misc.py Outdated
Comment thread src/sysdiagnose/utils/apollo.py
Comment thread src/sysdiagnose/utils/apollo.py
@cvandeplas
cvandeplas merged commit f997dc4 into main Jun 23, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[parsers|analysers] BaseInterface should contain a member containing the parsing/analysing errors

3 participants