-
Notifications
You must be signed in to change notification settings - Fork 5
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
init CI and some fixes #10
base: master
Are you sure you want to change the base?
Conversation
fix dockerfile and vladdoster config, setup CI
This reverts commit 0712fa0.
3x8 fix ci
* remove caching * redo a pipeline * add matrix * remove fail-fast * rename a job Authored-by: tripleight <Denis>
lbin'!**/tree' Old-Man-Programmer/tree \ | ||
lbin'!**/zsd' $ZI_REPO/zshelldoc | ||
lbin'!**/tree*' Old-Man-Programmer/tree \ | ||
lbin'!**/zsd*' $ZI_REPO/zshelldoc |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
not sure if you've tried this, but this glob actually breaks zsd
and tree
.
lbin'!**/zsd*' $ZI_REPO/zshelldoc | |
lbin'!**/bin/zsd' $ZI_REPO/zshelldoc |
@@ -107,8 +107,8 @@ zi as'command' light-mode for \ | |||
pick'zunit' atclone'./build.zsh' @zunit-zsh/zunit | |||
#=== COMPILED PROGRAMS ================================ | |||
zi lucid make'PREFIX=$PWD install' nocompile for \ | |||
lbin'!**/tree' Old-Man-Programmer/tree \ | |||
lbin'!**/zsd' $ZI_REPO/zshelldoc | |||
lbin'!**/tree*' Old-Man-Programmer/tree \ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lbin'!**/tree*' Old-Man-Programmer/tree \ | |
lbin'!**/bin/tree' Old-Man-Programmer/tree \ |
@@ -97,7 +97,7 @@ zi from'gh-r' lbin'!' nocompile for \ | |||
lbin'!**/rg' @BurntSushi/ripgrep \ | |||
lbin atinit" | |||
alias ll='exa -al'; | |||
alias l='exa -blF'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍🏻
@@ -27,7 +27,8 @@ RUN sh -c "$(curl -fsSL https://raw.githubusercontent.com/zdharma/zinit/master/d | |||
|
|||
# Copy configs into home directory | |||
ARG FOLDER | |||
COPY --chown=user "${FOLDER}" /home/user | |||
ENV XDG_DATA_HOME /home/user/.local/share | |||
COPY --chown=user ./${FOLDER} /home/user |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
COPY --chown=user ./${FOLDER} /home/user | |
COPY --chown=user ./"$FOLDER" /home/user |
paths: | ||
- .github/workflows/test_configs.yml | ||
- Dockerfile | ||
- vladdoster/.zshrc |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Might as well just normalize the configuration file to be zshrc
or something. I'll leave that to your discretion. So the following suggestions work
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I did this in #11 :)
For now, it runs on the changes of files that I've tested and fixed.
When more zsh configs will be fixed, this file should be copied for every config in the repo, or something like https://github.com/dorny/paths-filter might be used to be DRY.
Didn't use buildx because it can't use the cache right now cc docker/build-push-action#558
cc @vladdoster