Skip to content

Split github workflows for lower latency, add ruff - #1156

Merged
copybara-service[bot] merged 1 commit into
mainfrom
split-workflows
Dec 11, 2024
Merged

Split github workflows for lower latency, add ruff#1156
copybara-service[bot] merged 1 commit into
mainfrom
split-workflows

Conversation

@rdyro

@rdyro rdyro commented Dec 10, 2024

Copy link
Copy Markdown
Collaborator

This PR

  • splits linting and doctests from the combined test.sh to provide a faster workflow iteration
    • without removing that action from test.sh
    • in the future this duplication can be removed
  • adds ruff check . for fast linting check (currently in addition to pylint)
  • build-and-test and doctests workflows now also depend on the linting success

@rdyro
rdyro requested review from fabianp and vroulet December 10, 2024 21:17

@fabianp fabianp left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

thanks Robert!

@copybara-service
copybara-service Bot merged commit 78fe20e into main Dec 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants