[PR #495] [MERGED] os.path replacement #4204

New issue

Closed

opened 2026-03-15 01:32:02 +03:00 by kerem · 0 comments

kerem commented

2026-03-15 01:32:02 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/ArchiveBox/ArchiveBox/pull/495
Author: @afreydev
Created: 9/30/2020
Status: ✅ Merged
Merged: 10/2/2020
Merged by: @cdvv7788

Base: master ← Head: pathlib

📝 Commits (10+)

4f29db3 Replaced os.path in server.py
4ff7b66 Replaced os.path in init parsers
849820f Replaced os.path in schema.py
a0dbd80 Fixed string casting
91121d7 Replaced os.path in json.py
f1fd043 Replaced os.path in html.py
cd1b0bd Replaced os.path in init index
1eb0f44 Replaced os.path in favicon.py
6315506 Replaced od.path in init cli
9764a56 replaced os.path in init extractors

📊 Changes

14 files changed (+65 additions, -69 deletions)

View changed files

📝 archivebox/cli/__init__.py (+2 -1)
📝 archivebox/cli/tests.py (+9 -8)
📝 archivebox/config/__init__.py (+6 -6)
📝 archivebox/core/settings.py (+8 -7)
📝 archivebox/extractors/__init__.py (+1 -1)
📝 archivebox/extractors/favicon.py (+1 -2)
📝 archivebox/index/__init__.py (+1 -1)
📝 archivebox/index/html.py (+12 -15)
📝 archivebox/index/json.py (+9 -9)
📝 archivebox/index/schema.py (+2 -3)
📝 archivebox/logging_util.py (+6 -6)
📝 archivebox/parsers/__init__.py (+2 -3)
📝 archivebox/system.py (+1 -1)
📝 tests/mock_server/server.py (+5 -6)

📄 Description

Summary

This PR replaces all os.path to pathlib Path class.

Changes these areas

Bugfixes
Feature behavior
Command line interface
Configuration options
Internal architecture
Archived data layout on disk

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/ArchiveBox/ArchiveBox/pull/495 **Author:** [@afreydev](https://github.com/afreydev) **Created:** 9/30/2020 **Status:** ✅ Merged **Merged:** 10/2/2020 **Merged by:** [@cdvv7788](https://github.com/cdvv7788) **Base:** `master` ← **Head:** `pathlib` --- ### 📝 Commits (10+) - [`4f29db3`](https://github.com/ArchiveBox/ArchiveBox/commit/4f29db3c1652eec0b587625dfa814f666ebf82b2) Replaced os.path in server.py - [`4ff7b66`](https://github.com/ArchiveBox/ArchiveBox/commit/4ff7b662dcee907e29556117ab36e0620793c580) Replaced os.path in init parsers - [`849820f`](https://github.com/ArchiveBox/ArchiveBox/commit/849820fabb161ebe1536b4ea53ff07b94f820c7c) Replaced os.path in schema.py - [`a0dbd80`](https://github.com/ArchiveBox/ArchiveBox/commit/a0dbd80d3dd27bde5c5729742983304d2c83a1b8) Fixed string casting - [`91121d7`](https://github.com/ArchiveBox/ArchiveBox/commit/91121d776255546da91cb0068f7aef8c9066bd2a) Replaced os.path in json.py - [`f1fd043`](https://github.com/ArchiveBox/ArchiveBox/commit/f1fd0433b0d62b8ac090a6c1c14dbb4cc7eb2f6d) Replaced os.path in html.py - [`cd1b0bd`](https://github.com/ArchiveBox/ArchiveBox/commit/cd1b0bd22b2a2158d8501b6f14881d5e9a4af294) Replaced os.path in init index - [`1eb0f44`](https://github.com/ArchiveBox/ArchiveBox/commit/1eb0f44a60b40ca78757a320808a53d9381eb208) Replaced os.path in favicon.py - [`6315506`](https://github.com/ArchiveBox/ArchiveBox/commit/6315506bc3040f197942f1e27fab4541ba03b960) Replaced od.path in init cli - [`9764a56`](https://github.com/ArchiveBox/ArchiveBox/commit/9764a562f665f925995df9b0b07168bda3b09b7c) replaced os.path in init extractors ### 📊 Changes **14 files changed** (+65 additions, -69 deletions) <details> <summary>View changed files</summary> 📝 `archivebox/cli/__init__.py` (+2 -1) 📝 `archivebox/cli/tests.py` (+9 -8) 📝 `archivebox/config/__init__.py` (+6 -6) 📝 `archivebox/core/settings.py` (+8 -7) 📝 `archivebox/extractors/__init__.py` (+1 -1) 📝 `archivebox/extractors/favicon.py` (+1 -2) 📝 `archivebox/index/__init__.py` (+1 -1) 📝 `archivebox/index/html.py` (+12 -15) 📝 `archivebox/index/json.py` (+9 -9) 📝 `archivebox/index/schema.py` (+2 -3) 📝 `archivebox/logging_util.py` (+6 -6) 📝 `archivebox/parsers/__init__.py` (+2 -3) 📝 `archivebox/system.py` (+1 -1) 📝 `tests/mock_server/server.py` (+5 -6) </details> ### 📄 Description # Summary This PR replaces all os.path to pathlib Path class. # Changes these areas - [ ] Bugfixes - [ ] Feature behavior - [ ] Command line interface - [ ] Configuration options - [x] Internal architecture - [ ] Archived data layout on disk --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

kerem

2026-03-15 01:32:02 +03:00

closed this issue
added the
pull-request
label

No milestone

No project

No assignees

1 participant

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

starred/ArchiveBox#4204

No description provided.

Rows
Columns