Skip to content

(misc): code improvements#325

Closed
ss-o wants to merge 8 commits into
mainfrom
fix-variables
Closed

(misc): code improvements#325
ss-o wants to merge 8 commits into
mainfrom
fix-variables

Conversation

@ss-o

@ss-o ss-o commented Jun 15, 2025

Copy link
Copy Markdown
Member

This pull request includes updates to workflows, documentation, and Zsh scripts to improve functionality, maintainability, and compliance with best practices. Below is a summary of the most important changes, grouped by theme.

Workflow and CI Enhancements:

  • Updated .github/workflows/zunit.yml to remove the pull_request trigger, add a new env variable (branch_ref), and dynamically clone the repository using the current branch reference instead of hardcoding the main branch. [1] [2]

Documentation Updates:

  • Refactored docs/README.md to improve HTML structure, update image references, and enhance accessibility by replacing inline styles with standardized attributes (e.g., width and height). [1] [2] [3]

Zsh Script Improvements:

  • Improved lib/zsh/git-process-output.zsh by introducing a color configuration (ZI_USR[GP_COL]), optimizing the timeline() function, and adding compatibility with 24-bit color terminals. [1] [2] [3] [4]
  • Enhanced lib/_zi by adding a new version command and refining the unload and report logic to handle additional plugin states. [1] [2] [3]

These changes collectively improve the project's workflows, documentation clarity, and Zsh script functionality while adhering to best practices.

ss-o and others added 3 commits August 8, 2023 11:22
…aliases and set owner of packages.

Signed-off-by: Salvydas Lukosius <sall@w-ss.io>
…sudo from path declaration in zi.zsh

Signed-off-by: Salvydas Lukosius <ss-o@users.noreply.github.com>
Signed-off-by: Salvydas Lukosius <ss-o@users.noreply.github.com>
@ss-o
ss-o requested a review from a team as a code owner June 15, 2025 15:40
ss-o and others added 5 commits June 15, 2025 16:41
* [refactor] Update codebase (#264)
* [feat]: ZI_USR hash
* [fix]: updates/deprecates/adds subcommand options and other minor helpers.
* [refactor]: ⚡️ rename zi-show-times > zi-times + minor fixes

---------



















* message/output (#265)
[feat]: improve/rewrite/silence message.
* updates (#266)
refactor: ⚡️ improves updates
* zpmod/build (#267)
* [build]: ⚡️ consolidate zpmod

---------

* chore: 🚀

* style: 🎨 add space in output of 'zi times'
* [fix]: .zi-prepare-home use builtin cd -q
* fix: 🐛 register subcommand: report
* improve .zi-module build process
* perf: ⚰️ deduplicate compinit calls

---------

Signed-off-by: Salvydas Lukosius <ss-o@users.noreply.github.com>
Co-authored-by: Salvydas Lukosius <ss-o@users.noreply.github.com>
Signed-off-by: Salvydas Lukosius <ss-o@users.noreply.github.com>
perf: ⚡️ Checks the cached .zcompdump file to see if it must be regenerated adds a noticable delay to zsh startup.

This restricts checking it once a day, determines when to regenerate compinit (it doesn't need to modify the compdump everytime) and compiles mapped to share (total mem reduction) run in background in multiple shells. A function that can be invoked from within `atinit', `atload', e.g: zi ice atinit'zicompinit_fast'; zi load user/repository.

Signed-off-by: Salvydas Lukosius <sall@w-ss.io>
Co-authored-by: Salvydas Lukosius <ss-o@users.noreply.github.com>
@ss-o ss-o linked an issue Jun 20, 2025 that may be closed by this pull request
3 tasks
@ss-o ss-o removed a link to an issue Jun 20, 2025
3 tasks
@github-actions

Copy link
Copy Markdown

There hasn't been any activity on this pull request recently, and in order to prioritize active work, it has been marked as stale.
This PR will be closed and locked in 7 days if no further activity occurs.
Thank you for your contributions!

@ss-o ss-o removed the stale 👻 label Jul 22, 2025
@ss-o
ss-o marked this pull request as draft July 22, 2025 23:19
@github-actions

Copy link
Copy Markdown

There hasn't been any activity on this pull request recently, and in order to prioritize active work, it has been marked as stale.
This PR will be closed and locked in 7 days if no further activity occurs.
Thank you for your contributions!

@github-actions

github-actions Bot commented Sep 7, 2025

Copy link
Copy Markdown

Pull Request closed and locked due to lack of activity.
If you'd like to build on this closed PR, you can clone it using this method: https://stackoverflow.com/a/14969986
Then open a new PR, referencing this closed PR in your message.

@github-actions github-actions Bot locked and limited conversation to collaborators Sep 7, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant