Skip to content

Make lint mandatory for integration and deploy_website#7310

Merged
friedrichg merged 3 commits into
masterfrom
make-lint-mandatory
Mar 4, 2026
Merged

Make lint mandatory for integration and deploy_website#7310
friedrichg merged 3 commits into
masterfrom
make-lint-mandatory

Conversation

@friedrichg
Copy link
Copy Markdown
Member

Makes lint mandatory for integration.

This should make it easy to spot lint bugs. As they will not allow integration tests to run without it

Signed-off-by: Friedrich Gonzalez <1517449+friedrichg@users.noreply.github.com>
@dosubot dosubot Bot added the ci/cd label Mar 2, 2026
Signed-off-by: Friedrich Gonzalez <1517449+friedrichg@users.noreply.github.com>
This reverts commit dcf5985.

Signed-off-by: Friedrich Gonzalez <1517449+friedrichg@users.noreply.github.com>
@friedrichg friedrichg requested a review from SungJin1212 March 4, 2026 08:31
Copy link
Copy Markdown
Member

@SungJin1212 SungJin1212 left a comment

Choose a reason for hiding this comment

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

LGTM

@dosubot dosubot Bot added the lgtm This PR has been approved by a maintainer label Mar 4, 2026
@friedrichg friedrichg merged commit 179f94e into master Mar 4, 2026
182 of 188 checks passed
@friedrichg friedrichg deleted the make-lint-mandatory branch March 4, 2026 12:24
Shvejan pushed a commit to Shvejan/cortex that referenced this pull request Mar 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci/cd lgtm This PR has been approved by a maintainer size/XS

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants