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

Clean Tools Module #3990

Open
BenediktMehl opened this issue Mar 18, 2025 · 2 comments · May be fixed by #4005
Open

Clean Tools Module #3990

BenediktMehl opened this issue Mar 18, 2025 · 2 comments · May be fixed by #4005
Assignees
@BenediktMehl
Labels
pr-analysis Issues that touch the analysis pr(oject). tech For technical stories without user impact (=refactoring stories).

Comments

@BenediktMehl
Copy link
Collaborator

BenediktMehl commented Mar 18, 2025 •
edited
Loading

Feature request

Description

As a developer, I want the tools module to only contain "Tools" that the user sees as tools so that the project structure stays clean.

Acceptance criteria

  • Misplaced modules like ccsh, Inquirer, ... (everythin except ValidationTool and InspectionTool) are moved into a new or different module (e.g. "main")
  • Why is there an interactive parser in the ccsh module and not in the interactiveParser module

Assumptions & Exclusions

Development notes (optional Task Breakdown)

  • [ ]
  • [ ]
  • [ ]

Open questions
@BenediktMehl BenediktMehl added pr-analysis Issues that touch the analysis pr(oject). tech For technical stories without user impact (=refactoring stories). labels Mar 18, 2025
@BenediktMehl BenediktMehl self-assigned this Mar 21, 2025
BenediktMehl added a commit that referenced this issue Mar 21, 2025
@BenediktMehl
docs(analysis): add ADR (#3990)
34a9efc
BenediktMehl added a commit that referenced this issue Mar 21, 2025
@BenediktMehl
docs(analysis): write ADR (#3990)
f7ab7f1
@BenediktMehl
Copy link
Collaborator Author

Talk with team about the ADR

BenediktMehl added a commit that referenced this issue Mar 24, 2025
@BenediktMehl
docs(analysis): accept ADR (#3990)
3e0bbe9
@BenediktMehl BenediktMehl added the blocked The issue or feature is blocked on another issue or PR that has to be resolved first label Mar 25, 2025
@BenediktMehl
Copy link
Collaborator Author

Do this after #4000

@BenediktMehl BenediktMehl removed the blocked The issue or feature is blocked on another issue or PR that has to be resolved first label Mar 25, 2025
BenediktMehl added a commit that referenced this issue Mar 25, 2025
@BenediktMehl
docs(analysis): add ADR (#3990)
c50bc23
BenediktMehl added a commit that referenced this issue Mar 25, 2025
@BenediktMehl
docs(analysis): write ADR (#3990)
67a4b84
BenediktMehl added a commit that referenced this issue Mar 25, 2025
@BenediktMehl
docs(analysis): accept ADR (#3990)
df42bfc
BenediktMehl added a commit that referenced this issue Mar 25, 2025
@BenediktMehl
docs(analysis): remove MG from ADR (#3990)
0593d30
LisaWalter90 pushed a commit that referenced this issue Mar 28, 2025
refactor(analysis): move tools (#3990)
3bc1e81
LisaWalter90 pushed a commit that referenced this issue Mar 28, 2025
test(analysis): fix unit test (#3990)
96d956c
LisaWalter90 pushed a commit that referenced this issue Mar 28, 2025
test(analysis): fix unit test (#3990)
06e861c
LisaWalter90 pushed a commit that referenced this issue Mar 28, 2025
refactor(analysis): move export (#3990)
de13d9c
LisaWalter90 pushed a commit that referenced this issue Mar 28, 2025
refactor(analysis): rename exporters (#3990)
7ba1863
LisaWalter90 pushed a commit that referenced this issue Mar 28, 2025
refactor(analysis): rename analysers.tools package (#3990)
1c1590c
LisaWalter90 pushed a commit that referenced this issue Mar 28, 2025
refactor(analysis): remove redundant qualifiers (#3990)
4614f12
@BenediktMehl BenediktMehl linked a pull request Mar 28, 2025 that will close this issue
Open
6 tasks
LisaWalter90 pushed a commit that referenced this issue Mar 28, 2025
refactor(analysis): move filter (#3990)
8c66822
LisaWalter90 pushed a commit that referenced this issue Mar 28, 2025
refactor(analysis): remove redundant companion reference (#3990)
e424424
LisaWalter90 pushed a commit that referenced this issue Mar 28, 2025
refactor(analysis): rename filters (#3990)
9c1f4f0
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@BenediktMehl BenediktMehl

Labels
pr-analysis Issues that touch the analysis pr(oject). tech For technical stories without user impact (=refactoring stories).
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Tech/3990/clean tools module MaibornWolff/codecharta
1 participant
@BenediktMehl