Skip to content

Only test translation notebooks if they have been changed#3064

Merged
karlhigley merged 4 commits intoOpenMined:masterfrom
nph4rd:fix-translation-tests
Feb 21, 2020
Merged

Only test translation notebooks if they have been changed#3064
karlhigley merged 4 commits intoOpenMined:masterfrom
nph4rd:fix-translation-tests

Conversation

@nph4rd
Copy link
Member

@nph4rd nph4rd commented Feb 18, 2020

#3060 - Second try.

Solves the same problem as #3059 but only tests the notebooks if they have actually been changed.

The following command changed:

git diff --name-only origin/${{ github.base_ref }} examples/tutorials/translations/ > ./test/notebooks/git-diff.txt

Hopefully that will do the job.

@nph4rd nph4rd requested a review from karlhigley February 18, 2020 16:09
@karlhigley karlhigley added Type: Refactor 🔨 A complete overhaul of a file, feature, or codebase Type: Testing 🧪 Add testing or improving existing testing of a file, feature, or codebase translation labels Feb 19, 2020
@nph4rd nph4rd force-pushed the fix-translation-tests branch from 9987d61 to 8f51113 Compare February 19, 2020 22:47
@nph4rd nph4rd force-pushed the fix-translation-tests branch from b0e392a to 2c7f764 Compare February 20, 2020 23:40
@karlhigley karlhigley merged commit 4df727d into OpenMined:master Feb 21, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Type: Refactor 🔨 A complete overhaul of a file, feature, or codebase Type: Testing 🧪 Add testing or improving existing testing of a file, feature, or codebase

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants