Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Resolve minor issues with satellite template #2

Merged
merged 2 commits into from
Aug 26, 2020
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
13 changes: 10 additions & 3 deletions workflow-templates/cfgov-python-satellite-ci.yml
Original file line number Diff line number Diff line change
@@ -1,6 +1,11 @@
name: Python CI workflow

on: [pull_request, push]
on:
push:
branches:
- master
- main
pull_request:

jobs:
backend:
Expand All @@ -10,8 +15,8 @@ jobs:
matrix:
# Add the appropriate Tox environments for the satellite here
toxenv:
- py36
- py38
- py36-dj22
- py38-dj22
include:
# Adjust the Python versions required for the tox environments as
# needed
Expand All @@ -22,6 +27,8 @@ jobs:

steps:
- uses: actions/checkout@v2
with:
fetch-depth: 0
Comment on lines +30 to +31

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This change was necessary due to a change in checkout@v2 that defaults to shallow cloning with no tags. I have opened a feature request issue to ask for the option to shallow clone but also pull down tags.


- name: Set up Python
uses: actions/setup-python@v1
Expand Down
4 changes: 3 additions & 1 deletion workflow-templates/cfgov-python-satellite-release.yml
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,9 @@ jobs:
release:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v1
- uses: actions/checkout@v2
with:
fetch-depth: 0

# If the package has a front-end build we need to set up and install
# Node and other dependencies
Expand Down
4 changes: 3 additions & 1 deletion workflow-templates/cfpb-pypi-publish.yml
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,9 @@ jobs:
runs-on: ubuntu-latest
steps:

- uses: actions/checkout@v1
- uses: actions/checkout@v2
with:
fetch-depth: 0

- name: Set up Python
uses: actions/setup-python@v1
Expand Down
2 changes: 2 additions & 0 deletions workflow-templates/cfpb-python-library-ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -22,6 +22,8 @@ jobs:

steps:
- uses: actions/checkout@v2
with:
fetch-depth: 0

- name: Set up Python
uses: actions/setup-python@v1
Expand Down