Skip push to main branch check in test workflow push

[marcelveldt]

Our test workflow should ignore the pre-commit action for push to main if running from CI (but still run locally)

[MartinHjelmare]

I don't think we should do this. Tests should run on main branch too.

[MartinHjelmare]

I think we should exclude that hook from pre-commit run on CI instead.

[marcelveldt]

I don't think we should do this. Tests should run on main branch too.

But then its duplicated ?
As it will run when you create the PR, run when you push to that PR's branch.

We never push to main directly (without a PR) because we have protected the branch as well as the pre-commit that prevents it.

[marcelveldt]

I want to prevent this from happening:
https://github.com/home-assistant-libs/python-matter-server/actions/runs/7995064218/job/21834533096

[marcelveldt]

I think we should exclude that hook from pre-commit run on CI instead.

That sounds good, do you have any idea how ?

[MartinHjelmare]

This should work:

SKIP=no-commit-to-branch pre-commit run --all-files

[MartinHjelmare]

At least it works locally.

[marcelveldt]

pre-commit/pre-commit#1049

[marcelveldt]

@MartinHjelmare I have adjusted it - so now it excludes that rule only in the CI action but it will still run locally.

Also updated the PR description.

[MartinHjelmare]

Great!