Skip to content
This repository has been archived by the owner on Aug 2, 2022. It is now read-only.

Tilesets API #236

Closed
wants to merge 19 commits into from
Closed

Tilesets API #236

wants to merge 19 commits into from

Conversation

critical-path
Copy link
Contributor

This attempts to address #152. I would be happy to make changes to the code, tests, etc. Thank you!

@sgillies
Copy link
Contributor

@critical-path there are a few conflicts to resolve after the merger of #235. My schedule is full for the rest of this week, but I think I'll be able to review this next week.

@critical-path
Copy link
Contributor Author

@sgillies

I have resolved the merge conflict. Please let me know if you would like additional changes. Thank you!

@sgillies
Copy link
Contributor

Thanks @critical-path ! Tilesets API is the most crucial of these PRs to review, I think, unless you have other priorities.

@critical-path
Copy link
Contributor Author

critical-path commented Jul 10, 2018

@sgillies

Thank you for the opportunity to contribute to this project! My apologies if I tagged you too many times - I am still learning GitHub workflows and conventions.

I defer to you on prioritization. It seems very reasonable to review Tilesets first. Thank you!

@critical-path critical-path deleted the tilesets-api branch January 8, 2019 02:01
This was referenced Jul 30, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants