Skip to content

Conversation

@egmontkob
Copy link
Contributor

@egmontkob egmontkob commented Jan 12, 2026

This is to catch up with mc MidnightCommander/mc#4954 & MidnightCommander/mc#4955.

(Just a search-replace. Haven't tested.)

@egmontkob egmontkob requested a review from zyv January 12, 2026 11:47
Copy link
Member

@zyv zyv left a comment

Choose a reason for hiding this comment

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

👍

@egmontkob
Copy link
Contributor Author

A commit to which repo regenerates the skin homepage? (In other words: which one should I merge first?)

@zyv
Copy link
Member

zyv commented Jan 12, 2026

A commit to which repo regenerates the skin homepage? (In other words: which one should I merge first?)

Actually, both:

on:
push:
branches:
- master
repository_dispatch:
types:
- deploy-skins
workflow_dispatch:

I wouldn't fret too much over it, as long as the PRs are merged not too long after each other.

This is to catch up with mc #4954 / #4955.

Signed-off-by: Egmont Koblinger <[email protected]>
@egmontkob egmontkob force-pushed the skin-lowercase-lines-section branch from 44e7482 to fbaabb5 Compare January 18, 2026 08:55
@egmontkob egmontkob merged commit 0870686 into MidnightCommander:master Jan 18, 2026
2 checks passed
@egmontkob egmontkob deleted the skin-lowercase-lines-section branch January 18, 2026 08:58
@egmontkob
Copy link
Contributor Author

The skin homepage is up and running with lowercase [lines] sections.

This one is not backwards compatible; if you paste there a skin with uppercase [Lines] then it doesn't display properly. Honestly it's a "meh" from me, I don't think it's worth fixing.

@zyv
Copy link
Member

zyv commented Jan 18, 2026

This one is not backwards compatible; if you paste there a skin with uppercase [Lines] then it doesn't display properly. Honestly it's a "meh" from me, I don't think it's worth fixing.

Agreed.

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