Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@dpassen
Copy link
Contributor

@dpassen dpassen commented Dec 7, 2025

Checklist

  • The modified code has some test-coverage (where applicable).
  • make test is passing (this is what CI runs).
  • New unreleased features/fixes/styling and performance improvements are documented via git: feat: / fix: / style: / perf:. Or e.g. perf(highlighting):.
    See https://github.com/altsem/gitu/blob/master/docs/dev-tooling.md

@dpassen dpassen force-pushed the dpassen/push-lxtuvptmnrxl branch 2 times, most recently from f441d57 to 176d987 Compare December 7, 2025 17:09
@dpassen dpassen marked this pull request as ready for review December 7, 2025 17:09
Copy link
Owner

@altsem altsem left a comment

Choose a reason for hiding this comment

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

Looks good overall! It's just the order of stuff in the menu, and auto-generated changelog that would be missing I think.

There's little commit convention section at https://github.com/altsem/gitu/blob/master/docs/dev-tooling.md, primarily to produce the changelog. You could squash / force push commits.

Here's how it looks like atm:

> git cliff --unreleased
 INFO  git_cliff > A new version of git-cliff is available: v2.9.1 -> v2.10.1
# Changelog

All notable changes to this project will be documented in this file.

## [unreleased]

### 🚀 Features

- Rename 'Toggle Section' to 'Fold/Unfold', make it context aware

<!-- generated by git-cliff -->

@dpassen dpassen force-pushed the dpassen/push-lxtuvptmnrxl branch from 176d987 to b5123ea Compare December 7, 2025 20:05
Copy link
Owner

@altsem altsem left a comment

Choose a reason for hiding this comment

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

Nice! I'll merge it in :)

@altsem altsem merged commit bf55c40 into altsem:master Dec 7, 2025
@dpassen dpassen deleted the dpassen/push-lxtuvptmnrxl branch December 7, 2025 20:14
@dpassen
Copy link
Contributor Author

dpassen commented Dec 7, 2025

Thanks!

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