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

DM-23718: Replace dots in file templates with underscore #235

Merged
merged 1 commit into from Mar 6, 2020

Conversation

timj
Copy link
Member

@timj timj commented Mar 3, 2020

Sometimes physical_filter definitions include spaces and it's
much easier to use a file name if spaces are not there.

@timj timj requested a review from kfindeisen March 3, 2020 20:05
@parejkoj
Copy link
Contributor

parejkoj commented Mar 6, 2020

@timj : you technically can't be a reviewer of this because you created it, but can you please take a look? I'll leave it to you to update the ticket and PR description.

We should probably squash these two commits together?

I'm concerned about any effects this may have downstream if there are templates in use that are only basename that have '.' in them.

@timj
Copy link
Member Author

timj commented Mar 6, 2020

You mean you are worried about a template that refers to the special . directory but has no file component at all? A template that has no file in it is going to break.

@parejkoj
Copy link
Contributor

parejkoj commented Mar 6, 2020

The other way around: a template that has no / in it at all (and thus no directory, just basename).

@timj
Copy link
Member Author

timj commented Mar 6, 2020

We can trap a template that ends in a /

@timj
Copy link
Member Author

timj commented Mar 6, 2020

This doesn’t affect existing files in data stores. Only future ones.

Copy link
Member Author

@timj timj left a comment

Choose a reason for hiding this comment

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

Looks good. Yes squash since we aren't doing the space replacement.

tests/test_templates.py Outdated Show resolved Hide resolved
@timj timj changed the title DM-23718: Replace spaces in file templates with underscore DM-23718: Replace dots in file templates with underscore Mar 6, 2020
@timj
Copy link
Member Author

timj commented Mar 6, 2020

@parejkoj use git commit --amend --author "..." to update the author of the commit you just squashed so it's not me.

Sometimes physical_filter definitions include periods which cause the file
extension handling code to break up the filename incorrectly.
@parejkoj parejkoj merged commit afd9e45 into master Mar 6, 2020
@timj timj deleted the tickets/DM-23718 branch April 22, 2020 22:00
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

3 participants