Skip to content
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

feat: Git | New triggers of git commands #70

Merged
merged 2 commits into from
May 5, 2024

Conversation

angelorpt
Copy link
Contributor

Adding new triggers of git commands.

@federico-terzi
Copy link
Contributor

@angelorpt Amazing, thanks! Could you please rename the git.yml file into package.yml? :) Otherwise it won't be a valid syntax

@smeech smeech self-assigned this May 2, 2024
@smeech
Copy link
Collaborator

smeech commented May 2, 2024

No cmd, script, shell or malicious code AFAICS, although I'm not familiar with git commands.
No response to Federico's request a year ago - presumably we could rename the file at this end prior to merge, if approved?

@AucaCoyan
Copy link
Member

Yes, also all triggers begin with git so you won't do git reset --hard accidentally

@AucaCoyan
Copy link
Member

CI failed, but because main is was broken at cfb9d85 , not because this PR was wrong.
Squashing!

@AucaCoyan AucaCoyan merged commit d0c6418 into espanso:main May 5, 2024
1 check failed
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.

4 participants