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

Jump to bottom

Add smoke test #8

Merged
merged 2 commits into from
Jun 9, 2022
Merged

Add smoke test #8

merged 2 commits into from
Jun 9, 2022

Conversation

l0b0
Copy link
Contributor

@l0b0 l0b0 commented May 10, 2022 •
edited
Loading

No description provided.

@l0b0 l0b0 changed the title Test/add smoke test Add smoke test May 10, 2022
@l0b0 l0b0 force-pushed the test/add-smoke-test branch 6 times, most recently from 20ab5c9 to 3ad07ad Compare May 12, 2022 04:18
@l0b0 l0b0 force-pushed the test/add-smoke-test branch 2 times, most recently from 8a51d1a to 16c6c02 Compare June 2, 2022 00:49
@l0b0 l0b0 marked this pull request as ready for review June 2, 2022 00:50
@l0b0 l0b0 force-pushed the test/add-smoke-test branch from 16c6c02 to 702c33a Compare June 2, 2022 00:55
@l0b0 l0b0 requested a review from mfwightman June 2, 2022 00:56
@l0b0 l0b0 force-pushed the test/add-smoke-test branch from 702c33a to f166257 Compare June 2, 2022 03:43
@l0b0 l0b0 force-pushed the test/add-smoke-test branch from f166257 to 9060b22 Compare June 2, 2022 03:52
@l0b0 l0b0 added the automerge label Jun 2, 2022
@l0b0 l0b0 force-pushed the test/add-smoke-test branch 6 times, most recently from d98f89d to 66aa6bd Compare June 7, 2022 02:06
@l0b0
test: Run with Poetry
72e05ac 
On all supported Python versions. Python 3.10 did *not* work because of
an error building rasterio:

> rasterio/_warp.cpp:43950:19: error: ‘_PyGen_Send’ was not declared in
> this scope; did you mean ‘_PyGen_yf’?

Need to run on a specific Ubuntu version to avoid GDAL version drift.
@l0b0 l0b0 force-pushed the test/add-smoke-test branch from 66aa6bd to 72e05ac Compare June 7, 2022 03:47
@mfwightman mfwightman merged commit fd12de7 into master Jun 9, 2022
@l0b0 l0b0 deleted the test/add-smoke-test branch June 9, 2022 01:17
This was referenced Oct 27, 2022
Closed
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mfwightman mfwightman mfwightman approved these changes

Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@l0b0 @mfwightman