Skip to content

ci(python): run lint / unit tests / docs as GH actions#1333

Merged
parthea merged 5 commits intomasterfrom
add-lint-unittest-docs-checks-as-github-actions
Jan 20, 2022
Merged

ci(python): run lint / unit tests / docs as GH actions#1333
parthea merged 5 commits intomasterfrom
add-lint-unittest-docs-checks-as-github-actions

Conversation

@parthea
Copy link
Copy Markdown
Contributor

@parthea parthea commented Jan 17, 2022

I tested the changes using the following commands after cloning synthtool

git checkout add-lint-unittest-docs-checks-as-github-actions
docker build -f docker/owlbot/python/Dockerfile -t movetoghactions .

Once the docker image was built, I ran the following commands in the root of python-access-approval.

docker run --user $(id -u):$(id -g) --rm -v $(pwd):/repo -w /repo movetoghactions

This should not impact the existing kokoro or docs pre-submit jobs as we are adding github action checks in parallel. Once this is merged, we can disable the kokoro/docs on a per repo basis.

@parthea parthea added the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Jan 17, 2022
@parthea parthea changed the title ci: run lint / unit tests / docs as GH actions ci(python): run lint / unit tests / docs as GH actions Jan 17, 2022
@parthea parthea force-pushed the add-lint-unittest-docs-checks-as-github-actions branch from aaa14d8 to 27494be Compare January 17, 2022 20:48
@parthea parthea removed the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Jan 19, 2022
@parthea parthea marked this pull request as ready for review January 19, 2022 10:24
@parthea parthea requested a review from a team January 19, 2022 10:24
@parthea parthea added the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Jan 19, 2022
@parthea
Copy link
Copy Markdown
Contributor Author

parthea commented Jan 19, 2022

Wait for EOD to merge to avoid hitting kokoro github quota

@parthea parthea removed the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Jan 20, 2022
@parthea parthea merged commit 57be0cd into master Jan 20, 2022
@parthea parthea deleted the add-lint-unittest-docs-checks-as-github-actions branch January 20, 2022 02:23
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.

3 participants