Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Tech/3990/clean tools module #4005

Open
wants to merge 28 commits into
base: main
Choose a base branch
from
Open

Tech/3990/clean tools module #4005

wants to merge 28 commits into from

Conversation

BenediktMehl
Copy link
Collaborator

Big refactoring

Closes: #3990

Description

Descriptive pull request text, answering:

  • What problem/issue are you fixing?
  • What does this PR implement and how?

Definition of Done

A PR is only ready for merge once all the following acceptance criteria are fulfilled:

  • Changes have been manually tested
  • All TODOs related to this PR have been closed
  • There are automated tests for newly written code and bug fixes
  • All bugs discovered while working on this PR have been submitted as issues (if not already an open issue)
  • Documentation (GH-pages, analysis/visualization READMEs, parser READMEs, --help, etc.) has been updated (almost always necessary except for bug fixes)
  • CHANGELOG.md has been updated

Screenshots or gifs

@sonarqubecloud SonarQubeCloud
Copy link

sonarqubecloud bot commented Apr 1, 2025

Quality Gate Passed Quality Gate passed for 'CodeCharta Analysis'

Issues
52 New issues
0 Accepted issues

Measures
0 Security Hotspots
91.2% Coverage on New Code
2.0% Duplication on New Code

See analysis details on SonarQube Cloud

BenediktMehl and others added 28 commits April 1, 2025 16:54
@BenediktMehl @Nereboss
docs(analysis): add ADR (#3990)
7421fe6
@BenediktMehl @Nereboss
docs(analysis): write ADR (#3990)
c701323
@BenediktMehl @Nereboss
docs(analysis): accept ADR (#3990)
e797773
@BenediktMehl @Nereboss
docs(analysis): remove MG from ADR (#3990)
e7262da
@Nereboss
refactor(analysis): move tools (#3990)
16ad52c
@Nereboss
test(analysis): fix unit test (#3990)
36bdcb6
@Nereboss
test(analysis): fix unit test (#3990)
28da2a8
@Nereboss
refactor(analysis): move export (#3990)
860a69e
@Nereboss
refactor(analysis): rename exporters (#3990)
9f82c71
@Nereboss
refactor(analysis): rename analysers.tools package (#3990)
381ad9c
@Nereboss
refactor(analysis): remove redundant qualifiers (#3990)
9e13018
@Nereboss
refactor(analysis): move filter (#3990)
c4764c1
@Nereboss
refactor(analysis): remove redundant companion reference (#3990)
d41dbfc
@Nereboss
refactor(analysis): rename filters (#3990)
1a2669e
@Nereboss
refactor(analysis): move import (#3990)
751bec9
@Nereboss
refactor(analysis): rename importer (#3990)
cccec02
@Nereboss
refactor(analysis): move parser (#3990)
edcfd36
@Nereboss
refactor(analysis): rename parsers (#3990)
17ff916
@Nereboss
refactor(analysis): move InteractiveParser (#3990)
d0cb2a5
@Nereboss
refactor(analysis): rename AnalyserInterface (#3990)
ef71111
@Nereboss
refactor(analysis): move PipeableParser (#3990)
bbe2a9f
@Nereboss
refactor(analysis): rename PipeableAnalyserInterface (#3990)
e452f88
@Nereboss
refactor(analysis): move Inquirer (#3990)
6953b9f
@Nereboss
refactor(analysis): move ccsh (#3990)
4d131e7
@Nereboss
refactor(analysis): move DialogProvider (previously Inquirer) (#3990)
8562fe0
@Nereboss
refactor(analysis): rename DialogProvider (previously Inquirer) (#3990)
0622392
@Nereboss
refactor(analysis): update ADR (#3990)
e324400
@Nereboss
refactor(analysis): fix typo (#3990)
3763391
@Nereboss Nereboss force-pushed the tech/3990/clean-tools-module branch from 324edfd to 3763391 Compare April 1, 2025 14:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Clean Tools Module
2 participants
@BenediktMehl @Nereboss