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 patrol start time to get_patrol_events #350

Merged
merged 38 commits into from
Dec 4, 2024
Merged

Add patrol start time to get_patrol_events #350

merged 38 commits into from
Dec 4, 2024

Conversation

atmorling
Copy link
Contributor

Closes #349

@atmorling
flag io tests
0290ab0
@atmorling
Merge remote-tracking branch 'upstream/master' into test_overhaul
ea33cf4
@atmorling
tentative workflow rework
d1f6fd1
@atmorling
slightly less tentative
71228f3
@atmorling
no need to be doing er io in base tests
f84f629
@atmorling
proximity doesn't need to do er io
fc97056
@atmorling
fix filename typo
94f7098
@atmorling
forgot to remove skip
55eff95
@atmorling
param was lost in merge
b873b5d
@atmorling
needs a string wrapper
d2571f4
@atmorling
remove caching just to see what happens
4a40663
@atmorling
fail a test to verify workflow
a09b6fd
@atmorling
forgot to add mark
bdd3dd7
@atmorling
can revert test failure I think
de5a02c
@atmorling
fix syntax
ba74bba
@atmorling
inherit secrets
028100d
@atmorling
re-add env caching
a1a6372
@atmorling
is this even required
deb357c
@atmorling
-e
d0604ec
@atmorling
is pip the problem
78d3d48
@atmorling
revert- we need mamba for gdal
8ea30eb
@atmorling
schedule io tests
74d014a
@atmorling
rework os level tests
1ae5718
@atmorling
typo
896e66e
@atmorling
this isn't a var anymore
3505ffe
@atmorling
should cover io utils
a7e9dcd
@atmorling
add ee tools
796ef6b
@atmorling
seasons requires gee io
5c9cf91
@atmorling
remove skips
30316f3
@atmorling
now it's a seperate ci, let's just run io all the time
7ff8906
@cisaacstern
Copy link
Contributor

Sweet! Seems like base: should be #341?

@atmorling
Copy link
Contributor Author

Ah is that the trick to getting the commit history to reflect accurately. Does that just auto update to main once #341 goes in?

@cisaacstern
Copy link
Contributor

Does that just auto update to main once #341 goes in?

Correct. But can cause some issues with squash commits.

@atmorling atmorling changed the base branch from master to test_overhaul December 4, 2024 07:36
Base automatically changed from test_overhaul to master December 4, 2024 07:48
@atmorling atmorling merged commit 515d935 into master Dec 4, 2024
5 checks passed
@atmorling atmorling deleted the am/patrol_start_time branch December 4, 2024 13:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Add patrol_start_time to get_patrol_events response
2 participants
@atmorling @cisaacstern