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

BRN_ADM1 #2672 #2875

Merged
merged 1 commit into from
May 30, 2023
Merged

BRN_ADM1 #2672 #2875

merged 1 commit into from
May 30, 2023

Conversation

jingjli1
Copy link
Collaborator

@jingjli1 jingjli1 commented May 4, 2023

Why do we need this boundary?

Anything Unusual?

@jingjli1
Copy link
Collaborator Author

jingjli1 commented May 4, 2023

file upload for @rmbiddle

1 similar comment
@jingjli1
Copy link
Collaborator Author

jingjli1 commented May 4, 2023

file upload for @rmbiddle

@github-actions
Copy link

github-actions bot commented May 4, 2023

Hello! I am the geoBoundary Bot, and I just did an initial check on your submitted files.

I will automatically re-run my checks when you edit your Pull Request, and provide the new results in a comment just like this.
Once all of your files meet my programmed standards, I will flag your file for a manual human review.
=========== Submission Findings ===========

Full logs for fileChecks

Full logs for geometryDataChecks

Full logs for metaChecks

All checks have passed! I'll flag your boundary submission for a manual review by one of my humans.
Preview

lydiatroup
lydiatroup approved these changes May 4, 2023
View reviewed changes
Copy link
Collaborator

@lydiatroup lydiatroup left a comment

Choose a reason for hiding this comment

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

looks great! closes #2672

@DanRunfola DanRunfola merged commit ceb26a4 into main May 30, 2023
4 checks passed
@DanRunfola DanRunfola deleted the jli5 branch May 30, 2023 20:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lydiatroup lydiatroup lydiatroup approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@jingjli1 @lydiatroup @DanRunfola