Skip to content

blurb: replace spaces with underscores in news directory #499

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

Merged
merged 17 commits into from
Mar 6, 2024
Merged

blurb: replace spaces with underscores in news directory #499

Changes from 1 commit
Commits
Show all changes
17 commits
Select commit Hold shift + click to select a range
0367715
Remove unused code
hugovk Mar 28, 2023
535fc6a
Add tests
hugovk Mar 28, 2023
3a84a69
Add some type annotations
hugovk Mar 28, 2023
67869b3
Use dashes instead of spaces in paths
hugovk Mar 28, 2023
36bd99f
Don't install flit
hugovk Mar 29, 2023
66bc9e0
Build and install package, then test installed package
hugovk Mar 29, 2023
e69848d
Install build
hugovk Mar 29, 2023
e68c9fe
Install in place for easier test coverage
hugovk Mar 30, 2023
9bb59f4
Test on Python 3.12
hugovk Jul 23, 2023
53ca535
Remove type hints
hugovk Jul 23, 2023
f98656d
Use underscores instead of dashes for spaces
hugovk Jul 23, 2023
53cae52
Sort the *normalized* filenames for blurb release.
larryhastings Jul 23, 2023
8e53d4a
Add glob_blurbs test for sorting by filename
hugovk Jul 23, 2023
acc9e05
Change docstring grammar
hugovk Jul 26, 2023
22b23c9
Merge branch 'main' into no-spaces
hugovk Dec 26, 2023
3e0a0ba
Merge remote-tracking branch 'upstream/main' into no-spaces
hugovk Jan 17, 2024
f63fea1
Replace replace.replace with dict.get
hugovk Jan 17, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Change docstring grammar 
Co-authored-by: Éric <merwok@netwok.org>
  • Loading branch information
@hugovk @merwok
hugovk and merwok authored Jul 26, 2023
commit acc9e059b590e97260ff54bd65c1db2cb15ae11c
4 changes: 2 additions & 2 deletions blurb/blurb.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -111,14 +111,14 @@

def sanitize_section(section):
"""
Cleans up a section string, making it viable as a directory name.
Clean up a section string, making it viable as a directory name.
"""
return section.replace("/", "-").replace(" ", "_")
Copy link
Member

Choose a reason for hiding this comment

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

To make things more explicit/readable, this could use a dict similar to the one used below for unsanitize_section.
If new sections that require sanitation are added in the future the function won't work, but it will serve as a reminder to update both dicts.

Copy link
Member Author

Choose a reason for hiding this comment

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

Updated.



def sanitize_section_legacy(section):
"""
Cleans up a section string, making it viable as a directory name (allow spaces).
Clean up a section string, making it viable as a directory name (allow spaces).
"""
return section.replace("/", "-")

Expand Down