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

[jaeger-idl] Reproduce linting checks from the main repo #6570

Open
yurishkuro opened this issue Jan 20, 2025 · 1 comment
Open

[jaeger-idl] Reproduce linting checks from the main repo #6570

yurishkuro opened this issue Jan 20, 2025 · 1 comment
Labels
enhancement good first issue Good for beginners help wanted Features that maintainers are willing to accept but do not have cycles to implement

Comments

@yurishkuro
Copy link
Member

yurishkuro commented Jan 20, 2025 •
edited
Loading

Part of #6494.

Currently jaeger-idl only has code-gen linter. We need to reproduce all the linters that exist in the main repository

https://github.com/jaegertracing/jaeger/blob/main/.github/workflows/ci-lint-checks.yaml

We don't want to duplicate all the scripts though, make the workflow clone the scripts from the main repo before running.
@yurishkuro yurishkuro added good first issue Good for beginners help wanted Features that maintainers are willing to accept but do not have cycles to implement labels Jan 20, 2025
@dosubot dosubot bot added the enhancement label Jan 20, 2025
@yurishkuro yurishkuro mentioned this issue Jan 20, 2025
Closed
@danish9039
Copy link
Contributor

currently working on this

@danish9039 danish9039 mentioned this issue Jan 20, 2025
Open
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement good first issue Good for beginners help wanted Features that maintainers are willing to accept but do not have cycles to implement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@yurishkuro @danish9039