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.

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

Remove circular dependency in lib/core/module.ts > lib/controller/index.ts > lib/processors/validate-processor.ts #1631

Open
Tracked by #1244
samayer12 opened this issue Jan 6, 2025 · 0 comments
Assignees
Milestone

Comments

@samayer12
Copy link
Contributor

Describe what should be investigated or refactored

A circular dependency exists between these files. We should refactor our code so that it no longer exists.

It's possible that there will be a chain of related circular dependencies while working on this issue, try to scope work down to just the specified files if possible. If not possible, cross-reference to other circular dependency issues.

Links to any relevant code

N/A

Additional context

Run npx madge --circular --ts-config tsconfig.json --extensions ts,js src/ to check for circular dependencies locally.

@cmwylie19 cmwylie19 added this to the v0.44.0 milestone Jan 6, 2025
@cmwylie19 cmwylie19 moved this from 🆕 New to 📋 Backlog in Pepr Project Board Jan 6, 2025
@cmwylie19 cmwylie19 modified the milestones: v0.43.0, v0.44.0, v0.45.0 Jan 21, 2025
@samayer12 samayer12 self-assigned this Jan 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: 📋 Backlog
Development

No branches or pull requests

2 participants