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

aggregate_spatial_window typo fix #446

Merged
merged 1 commit into from
Jul 31, 2023
Merged

aggregate_spatial_window typo fix #446

merged 1 commit into from
Jul 31, 2023

Conversation

soxofaan
Copy link
Member

addresses minor typo in aggregate_spatial_window (and eliminate triple "to" sentence)

@m-mohr m-mohr changed the base branch from master to draft July 31, 2023 11:36
@m-mohr m-mohr changed the base branch from draft to master July 31, 2023 11:36
@m-mohr
Copy link
Member

m-mohr commented Jul 31, 2023

Thanks, but PRs must be made against the draft branch.

@soxofaan
Copy link
Member Author

oh right, I did this edit right from github.com webui and missed that bit.

I now force-pushed a rebase, but I'm not sure I should have done that as you apparently did a cleanup commit as well

@m-mohr m-mohr changed the base branch from master to draft July 31, 2023 11:50
@m-mohr
Copy link
Member

m-mohr commented Jul 31, 2023

Thanks!

No, the clean-up was coming from master, changing the base here makes it work. :-)

@m-mohr m-mohr self-requested a review July 31, 2023 11:51
@m-mohr m-mohr merged commit 7a29378 into Open-EO:draft Jul 31, 2023
1 check passed
@soxofaan soxofaan deleted the patch-4 branch August 7, 2023 13:45
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.

None yet

2 participants