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

mapfullgrid.sh no longer needed #142

Closed
wants to merge 1 commit into from
Closed

Conversation

jordandekraker
Copy link
Collaborator

As discussed in #128, testing is working well without this script.

TODO:
harmonize with #130

@akhanf
Copy link
Member

akhanf commented Dec 22, 2021

Excellent news! I'll have a look through this too to make sure its a clean break, and remove any extra files/resources/dependencies we don't need as well. Then we can merge this in, and I can update #130 accordingly.

@akhanf akhanf added the enhancement New feature or request label Dec 22, 2021
@akhanf
Copy link
Member

akhanf commented Dec 22, 2021

Actually, I think it will be easier to just drop mapfullgrid in #130.. I'll go ahead and do that..

@jordandekraker
Copy link
Collaborator Author

aha yeah i had already started on trying to merge the two branches. Definitely easier to do on a new branch though, i'll test out #143

@jordandekraker jordandekraker deleted the mapfullgrid branch January 6, 2022 18:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants