Skip to content

tombi: support toml_tombi_offline flag for offline usage#5032

Merged
hsanson merged 1 commit intodense-analysis:masterfrom
mathstuf:tombi-offline-option
Nov 22, 2025
Merged

tombi: support toml_tombi_offline flag for offline usage#5032
hsanson merged 1 commit intodense-analysis:masterfrom
mathstuf:tombi-offline-option

Conversation

@mathstuf
Copy link
Contributor


Cc: @rymdbar

@mathstuf mathstuf force-pushed the tombi-offline-option branch 2 times, most recently from 11ba2b1 to 3426b39 Compare August 19, 2025 02:24
@mathstuf mathstuf marked this pull request as ready for review August 19, 2025 02:30
Copy link
Contributor

@rymdbar rymdbar left a comment

Choose a reason for hiding this comment

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

Thanks for adding this! I found a minor detail or two.

@mathstuf mathstuf force-pushed the tombi-offline-option branch from 3426b39 to b022b4e Compare September 1, 2025 03:33
@mathstuf mathstuf requested a review from rymdbar September 2, 2025 17:35
@mathstuf mathstuf force-pushed the tombi-offline-option branch from b022b4e to 2f0c677 Compare September 2, 2025 19:11
Copy link
Contributor

@rymdbar rymdbar left a comment

Choose a reason for hiding this comment

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

I reckon this looks good!

My recommendation for @hsanson is to merge it.

Copy link
Contributor

@hsanson hsanson left a comment

Choose a reason for hiding this comment

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

Thanks for the contribution and @rymdbar thanks for the review.

@hsanson hsanson merged commit 86d8ada into dense-analysis:master Nov 22, 2025
8 checks passed
@mathstuf mathstuf deleted the tombi-offline-option branch November 23, 2025 21:31
Caw3 pushed a commit to Caw3/ale that referenced this pull request Dec 10, 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.

3 participants