nizarmah / auto-minify

Minifies JS and CSS files in GitHub workflows.

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Update readme examples

Spreeuw opened this issue Β· comments

The version constraints in the readme examples are a little dated or not ideal I would suggest to make these changes:

stefanzweifel/git-auto-commit-action@v3.0.0

to:

stefanzweifel/git-auto-commit-action@v4

and your own:

nizarmah/auto-minify@master

I would change to

nizarmah/auto-minify@v2

(I would consider master to be more like the bleeding edge for people who need quick access to an unreleased fix, but stable releases more suitable for most people)

commented

@Spreeuw you're right! Thanks for that! Back when I wrote the README, I included master without any regard to the importance of having actions not unexpectedly "break".

Regarding the git-auto-commit-action upgrade, sounds good as well πŸ‘πŸΌ Would you prefer to open a pull request with those changes and have me review it? Or would you like me to create a pull request, while handling the review? Let me know πŸ˜ƒ

I can also create a PR without your review, but a little bit of friction over PRs is beneficial for both parties involved πŸ˜‰

commented

Thanks for the review @Spreeuw and for the helpful suggestions πŸ˜ƒ I hope you have a nice day!