Skip to content

Update gitignore implementation to support more cases#2687

Closed
colinjc wants to merge 1 commit intodstackai:masterfrom
colinjc:cc/05-23-update_gitignore_implementation_to_support_more_cases
Closed

Update gitignore implementation to support more cases#2687
colinjc wants to merge 1 commit intodstackai:masterfrom
colinjc:cc/05-23-update_gitignore_implementation_to_support_more_cases

Conversation

@colinjc
Copy link
Copy Markdown
Contributor

@colinjc colinjc commented May 23, 2025

Existing implementation was missing support for a ** which was pretty critical. This updates the logic to use an existing library which already supports **.

@jvstme jvstme self-requested a review May 26, 2025 09:09
from pathlib import Path
from typing import BinaryIO, Optional

import humanize
Copy link
Copy Markdown
Collaborator

@r4victor r4victor May 26, 2025

Choose a reason for hiding this comment

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

Note that I dropped humanize dependency in #2692 (#2691) since for dstack purposes it can be replaced with a simple function.

Use from dstack._internal.utils.common import sizeof_fmt instead.

@colinjc colinjc force-pushed the cc/05-23-update_gitignore_implementation_to_support_more_cases branch from ec926a5 to de4bbf1 Compare May 26, 2025 13:38
@colinjc colinjc force-pushed the cc/05-23-update_gitignore_implementation_to_support_more_cases branch from de4bbf1 to d7fa66b Compare May 26, 2025 13:39
@colinjc colinjc closed this May 26, 2025
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