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

fix minor windows bugs #26

Draft
wants to merge 8 commits into
base: master
Choose a base branch
from
Draft

fix minor windows bugs #26

wants to merge 8 commits into from

Conversation

mbarison
Copy link

Minor bugfixes:

  • Remove quotes and double quotes from root dir (for pathnames with spaces)
  • Avoid double blank CSV lines on Windows

@mbarison mbarison requested a review from mneyrane January 24, 2023 22:31
@mbarison mbarison marked this pull request as draft January 25, 2023 15:40
@mneyrane
Copy link
Member

Hi @mbarison , I can review these changes later this week as I'm a bit busy.

Were you planning to include any further commits?

@mneyrane
Copy link
Member

Hey, apologies for the (notably) delayed response. I'm willing to pull in these changes provided the respective source file documentation is updated to include JSON and GeoJSON tags.

If you're unfamiliar with Sphinx, feel free to draft the changes in a text or Markdown file. From there, I can edit and incorporate the changes.

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.

2 participants