Skip to content

ci: only run pre-commit pyright on staged files - #1555

Open
tj098895 wants to merge 1 commit into
mainfrom
tj098895-patch-2
Open

ci: only run pre-commit pyright on staged files#1555
tj098895 wants to merge 1 commit into
mainfrom
tj098895-patch-2

Conversation

@tj098895

@tj098895 tj098895 commented May 13, 2026

Copy link
Copy Markdown
Contributor

Type of Work

  • Patch: X.Y.Z+1. NEGLIGIBLE visible changes, does not change input or output - OR changes behaviour. Use chore:, refactor: etc
  • Minor: X.Y+1.Z. Minor changes, might ADD new input (YAML), or other backwards-compatible changes. Use feat:, fix:
  • Major: X+1.Y.Z. Major and most likely BREAKING changes, wo. backwards compatibility, or removing temporary backwards compatibility functionality. Use ! or BREAKING:.

See here (internal): https://github.com/equinor/ecalc-internal/discussions/1044

Have you remembered and considered?

  • IF FEAT: I have remembered to update documentation
  • IF FIX OR FEAT: I have remembered to update manual changelog (docs/drafts/next.draft.md)
  • IF BREAKING: I have remembered to update migration guide (docs/docs/migration_guides/)
  • IF BREAKING: I have committed with BREAKING: in footer or ! in header
  • I have added tests (if not, comment why)
  • I have used conventional commits syntax (if you squash, make sure that conventional commit is used)
  • I have included the Github issue nr in the footer!

What is this PR all about?

What else did you consider?

Between the lines?

@tj098895
tj098895 requested a review from a team as a code owner May 13, 2026 19:15
@Aleksander-Karlsson

Copy link
Copy Markdown
Contributor

Is the errors real (and should be baselined) or is it just not finding the config files?

@github-actions

Copy link
Copy Markdown
Contributor

This PR is stale because it has been open for 7 days with no activity.

@github-actions github-actions Bot added the Stale label Aug 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants