Fix to add cpu architecture suffix to artifact name of cljstyle

DeLaGuardo / setup-clojure

GitHub Action to provision clojure's most popular build tools for Linux, Mac OS X and Windows.

MIT License

183 stars 27 forks source link

Fix to add cpu architecture suffix to artifact name of cljstyle #101

Closed hatappo closed 8 months ago

hatappo commented 8 months ago

To fix the issue: https://github.com/DeLaGuardo/setup-clojure/issues/100

Thank you for maintaining a great workflow like this repository. We always appreciate your work.

DeLaGuardo commented 8 months ago

LGTM! thanks)

Before merging, could you run npm run all and commit changed sources? This command will run a few tests and compile .ts to .js files.

hatappo commented 8 months ago

I did not take care of the test cases. My apologies.

I modified the test code as well to accommodate the logic changes and also committed the built artifacts. (Should I add a test case for boundary conditions by version number? If so, please let me know.)