GitHub CI. Run only relative platforms changes - #3395
GitHub CI. Run only relative platforms changes#3395Igor Demin (igordmn) wants to merge 4 commits into
Conversation
Igor Demin (igordmn)
changed the title
5ed255a to
54d805c
Compare
febfb62 to
54d805c
Compare
| - '.github/path-filter.yml' | ||
|
|
||
| # Change with sourceSetsUnknown | ||
| sourceSetsIos: |
There was a problem hiding this comment.
The idea of this PR is to run only iOS/Desktop/Web related workflows if we change only these folders, and run all the tests if change something else.
Ivan Matkov (@MatkovIvan), Alexander Maryanovsky (@m-sasha), Oleksandr Karpovich (@eymar), Andrei Salavei (@ASalavei), does it sound good to you?
There was a problem hiding this comment.
Sounds great to me!
There was a problem hiding this comment.
Asked you all to review. Ivan - the logic, the rest of you - the source set list
There was a problem hiding this comment.
the source set list
And the related platform workflow changes
| - '**/tvosTest/**' | ||
| - '**/watchosMain/**' | ||
| - '**/watchosTest/**' | ||
| - '**/linuxMain/**' |
There was a problem hiding this comment.
Do we need these 4 sources sets here?
There was a problem hiding this comment.
We have these source sets, or at least we had and can have in the future.
I renamed it to "...Native"
| - '**/darwinTest/**' | ||
| - '**/nativeMain/**' | ||
| - '**/nativeTest/**' | ||
| - '**/uikitMain/**' |
There was a problem hiding this comment.
these 2 are legacy - I would remove them
9fc36f5 to
6cb8b4d
Compare
09051eb to
99ade3d
Compare
3 participants
Run only Desktop/iOS/Web if only their source sets changed, otherwise run everything.
Reasons:
The full set of changes is still run after merging.
Testing
igor.demin/jb-main-base-tests-for-workflow-changes, changed the PR baseRelease Notes
N/A