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.

Sign up for GitHub

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

GitHub action update for preview and contribution workflow #87

Merged
merged 9 commits into from
Oct 30, 2023
Merged

GitHub action update for preview and contribution workflow #87

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
dbbf72c
Minor fix - added missing period
tylere Oct 27, 2023
20ee9f7
Add restriction to only run for PRs with main as the target.
zacdezgeo Oct 27, 2023
7d85fbf
Add check to CI enforce a branch restriction on PRs from non-staging …
zacdezgeo Oct 27, 2023
497765f
Separated check on PR for non-staging branch to another job
zacdezgeo Oct 27, 2023
fa302b1
Remove duplicate lines
zacdezgeo Oct 27, 2023
29b8f2b
Merge pull request #86 from cloudnativegeo/modify-preview-action
zacdezgeo Oct 30, 2023
9847e28
Merge branch 'staging' into patch-2
zacdezgeo Oct 30, 2023
cb4fe42
Merge pull request #85 from tylere/patch-2
zacdezgeo Oct 30, 2023
27c7a24
Merge branch 'main' into staging
zacdezgeo Oct 30, 2023
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
20 changes: 18 additions & 2 deletions .github/workflows/ci.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -8,16 +8,32 @@ on:
pull_request:

jobs:
pr-target-test:
runs-on: ubuntu-latest
steps:
- name: Fail if target is not staging (for PRs only)
if: github.event_name == 'pull_request'
run: |
if [[ "${{ github.event.pull_request.base.ref }}" != "main" || "${{ github.event.pull_request.head.ref }}" == "staging" ]]; then
echo "Target branch is acceptable."
else
echo "Only PRs from staging can target main."
exit 1
fi

tests:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v2

- name: Checkout code
uses: actions/checkout@v2

- name: Set up Python 3.11
uses: actions/setup-python@v2
with:
python-version: 3.11

- name: run pre-commit
- name: Run pre-commit
run: |
python -m pip install pre-commit
pre-commit run --all-files
2 changes: 2 additions & 0 deletions .github/workflows/preview.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,8 @@ name: Deploy PR previews

on:
pull_request:
branches:
- main
types:
- opened
- reopened
Expand Down
2 changes: 1 addition & 1 deletion index.qmd
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -59,7 +59,7 @@ Notes:
- GeoParquet and FlatGeobuf can be used for vector data or point cloud data.
- LAS files are intended for 3D points, not 2D points (since COPC files are compressed LAS files, the same goes for COPC files).
- [TopoJSON](https://github.com/topojson/topojson) (an extension of GeoJSON that encodes topology) and [newline-delimited GeoJSON](https://stevage.github.io/ndgeojson/) are types of GeoJSON worth mentioning but have yet to be explicitly represented in the diagram.
- GeoTIFF and GeoParquet are geospatial versions of the non-geospatial file formats TIFF and Parquet, respectively. FlatGeobuf builds upon the non-geospatial [flatbuffers](https://github.com/google/flatbuffers) serialization library (though flatbuffers is not a standalone file format)
- GeoTIFF and GeoParquet are geospatial versions of the non-geospatial file formats TIFF and Parquet, respectively. FlatGeobuf builds upon the non-geospatial [flatbuffers](https://github.com/google/flatbuffers) serialization library (though flatbuffers is not a standalone file format).

## Table of Contents

Expand Down
Loading