BlakeWilliams / Elixir-Slack

Slack real time messaging and web API client in Elixir
MIT License
674 stars 183 forks source link

Update GitHub actions workflow #253

Closed inooid closed 1 year ago

inooid commented 1 year ago

Description

For now I left mix credo out of it as it was not in the current pipeline and there's probably a lot to fix. Let's do this in future PRs. There's a lot of deprecated methods across versions and there's probably a discussion to be had over which OTP + Elixir versions we should support.

Note: This CI pipeline is inspired by the setup of: https://github.com/swoosh/swoosh

inooid commented 1 year ago

If someone has the chance to kick off the CI for this PR, I'd be very grateful 🙇 (or give me permission to kick it off myself as otherwise I'll have to continuously bother the maintainers on every push 😅)

inooid commented 1 year ago

For now I've made it do the same things as the current CI does. We can do follow up PRs cleaning up (e.g. deprecated methods, etc.) and adding credo and/or dialyxir. If necessary I can rebase and clean up the commits or we squash and merge depending on what's preferred (as there's no contribution document).

Feel free to review where you see fit 👍

BlakeWilliams commented 1 year ago

Thanks for fixing this up! ❤️