-
Notifications
You must be signed in to change notification settings - Fork 1
Refactor CI architecture and enhance documentation for ZUnit #63
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Changes from all commits
Commits
Show all changes
47 commits
Select commit
Hold shift + click to select a range
1208104
docs: add zd refactor design spec
ss-o 69a8311
docs: fix three spec gaps after self-review
ss-o d8d6b47
docs: add zd refactor implementation plan
ss-o ff5445a
feat: add scripts/ directory with build.sh and run.sh
ss-o 77653c4
fix: correct Dockerfile path, extra args, CONTAINER_ROOT, and DEVEL i…
ss-o 93b3f2f
feat: add tests/helpers.zsh with zi_test helper
ss-o 2260ef2
feat: add tests/setup.zsh and teardown.zsh (no-sudo, ZI_DATA based)
ss-o 09b7e18
feat: migrate annexes.zunit to zi_test helper
ss-o c63fe05
fix: add load helpers to annexes.zunit @setup block
ss-o 04f5a70
feat: migrate ice.zunit to zi_test helper
ss-o 08ad6c0
feat: migrate plugins.zunit to zi_test helper
ss-o d4f4acf
feat: migrate snippets.zunit to zi_test helper
ss-o 9df73bf
feat: migrate packages.zunit to zi_test helper
ss-o b216713
refactor: entrypoint.sh — user creation only, drop runtime downloads
ss-o 2432c26
refactor: Dockerfile — two-stage, zi pre-baked, ZSH_VERSION via zi pa…
ss-o 2a1193c
refactor: zshrc sources zi directly; drop prepare_system/initiate_system
ss-o cd4fbbd
fix: docker-compose context updated to repo root for COPY tests/ to work
ss-o bc88091
feat: add test-native.yml — native ZUnit CI without Docker
ss-o bcf0079
feat: add test-matrix.yml — Docker Zsh version matrix (scheduled)
ss-o c88e300
chore: remove docker/tests/, old scripts, and superseded zunit.yml wo…
ss-o 93807e0
feat: add Makefile for local testing and workflow_call to test-native…
ss-o 15db086
docs: add README and usage documentation
ss-o bc7723d
refactor: update configs, workflows, docs, and docker files
ss-o 630b4ae
Potential fix for pull request finding 'CodeQL / Workflow does not co…
ss-o 61e1c7b
Potential fix for pull request finding 'CodeQL / Workflow does not co…
ss-o 26ac278
fix: correct Docker build context, ARG name, and setup glob failure
ss-o 94bc528
fix: replace CDN install and zi-pack with direct git clone; pin Alpin…
ss-o 885986d
fix: remove packages absent in old Alpine; correct mv ice assertion
ss-o 3d090f1
Potential fix for pull request finding
ss-o adbacae
Potential fix for pull request finding
ss-o 9bb7b26
fix: make zunit clone step idempotent and keep clone errors visible
Copilot c8c198c
fix: restore full Zi init in zshrc and correct docs example path
Copilot 855aeac
tests: update failing mv ice assertion to match new hook error message
ss-o 808ff00
test: disable ANSI colors in zi_test subprocess
ss-o 20b4845
feat: migrate Docker base from Alpine to debian:trixie-slim
ss-o ba01f61
fix: chmod a+x for zunit/revolver/color so non-root user can execute
ss-o 11fece6
fix: resolve hadolint DL3003/DL3008/DL4001 trunk issues
ss-o afdecb2
fix: use wget for zsh source download; suppress DL4001
ss-o 5bbfed6
fix(docker): use git clone from GitHub to build zsh from source
ss-o be1dc2c
fix(docker): patch termcap.c for GCC 14 / ncurses 6 type conflict
ss-o a1a127c
fix(docker): rename termcap arrays to avoid ncurses symbol collision
ss-o e1d6a62
fix(docker): skip doc install; use install.bin/fns/modules
ss-o 70eaeff
fix: pin zdharma/zunit to v0.8.2 in Dockerfile and test-native.yml
ss-o 75a830c
fix(docker): add automake; update config.guess/sub before building ol…
ss-o cdf6bc4
fix(docker): use autoreconf --install --force for missing configure s…
ss-o 5b93b92
fix: arm64 config.guess + migrate revolver/color to z-shell/src
ss-o 6ed0a98
fix: quote variable interpolation in zi_test to prevent injection
Copilot File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,60 +1,27 @@ | ||
| # Space or Tabs? | ||
| # https://stackoverflow.com/questions/35649847/objective-reasons-for-using-spaces-instead-of-tabs-for-indentation | ||
| # https://stackoverflow.com/questions/12093748/how-to-use-tabs-instead-of-spaces-in-a-shell-script | ||
| # https://github.com/editorconfig/editorconfig-defaults/blob/master/editorconfig-defaults.json | ||
| # | ||
| # 1. What happens when I press the Tab key in my text editor? | ||
| # 2. What happens when I request my editor to indent one or more lines? | ||
| # 3. What happens when I view a file containing U+0009 HORIZONTAL TAB characters? | ||
| # | ||
| # Answers: | ||
| # | ||
| # 1. Pressing the Tab key should indent the current line (or selected lines) one additional level. | ||
| # 2. As a secondary alternative, I can also tolerate an editor that, | ||
| # like Emacs, uses this key for a context-sensitive fix-my-indentation command. | ||
| # 3. Indenting one or more lines should follow the reigning convention, if consensus is sufficiently strong; otherwise, | ||
| # I greatly prefer 2-space indentation at each level. U+0009 characters should shift subsequent characters to the next tab stop. | ||
| # | ||
| # Note: VIM users should use alternate marks [[[ and ]]] as the original ones can confuse nested substitutions, e.g.: ${${${VAR}}} | ||
| # | ||
| # -*- mode: zsh; sh-indentation: 2; indent-tabs-mode: nil; sh-basic-offset: 2; -*- | ||
| # vim: ft=zsh sw=2 ts=2 et | ||
|
|
||
| root = true | ||
|
|
||
| [*] | ||
| charset = utf-8 | ||
| indent_style = space | ||
| indent_size = 2 | ||
| end_of_line = lf | ||
| insert_final_newline = true | ||
| trim_trailing_whitespace = true | ||
|
|
||
| [*.sln] | ||
| indent_style = tab | ||
| indent_style = space | ||
| indent_size = 2 | ||
|
|
||
| [*.{md,mdx,rst}] | ||
| trim_trailing_whitespace = false | ||
|
|
||
| [*.{cmd,bat}] | ||
| end_of_line = crlf | ||
|
|
||
| [*za-*] | ||
| end_of_line = lf | ||
|
|
||
| [*.{sh,bash,zsh,fish}] | ||
| end_of_line = lf | ||
|
|
||
| [Makefile] | ||
| [Makefile*] | ||
| indent_style = tab | ||
| indent_size = 4 | ||
|
|
||
| [*.{py,rb}] | ||
| [*.py] | ||
| indent_size = 4 | ||
|
|
||
| [*.{go,java,scala,groovy,kotlin}] | ||
| [*.go] | ||
| indent_style = tab | ||
| indent_size = 4 | ||
|
|
||
| [*.{cs,csx,cake,vb,vbx}] | ||
| # Default Severity for all .NET Code Style rules below | ||
| dotnet_analyzer_diagnostic.severity = warning | ||
| [*.java] | ||
| indent_style = tab | ||
| indent_size = 4 | ||
Empty file.
This file was deleted.
Oops, something went wrong.
This file was deleted.
Oops, something went wrong.
This file was deleted.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file was deleted.
Oops, something went wrong.
This file was deleted.
Oops, something went wrong.
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.