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

resolve mix docs issue + dependency updates #44

Merged
merged 20 commits into from
Feb 6, 2024

Conversation

barrieloydall
Copy link
Contributor

@barrieloydall barrieloydall commented Feb 3, 2024

Started off fixing issue of unable to build docs on elixir:1.15.7, but made a bunch of other updates that can be included in the 0.4.0 release

  • add mix docs to git workflow, to ensure docs build after changes
  • remove unused earmark dep
  • the issue with building docs on 1.15.7 was due to the unnecessary addition of the applications: config. Should not be required as those "apps" are defined in deps()
  • dependency updates
  • credo update and fixes
  • update docs and typos
  • add dependabot and codeowner configs

@barrieloydall barrieloydall marked this pull request as ready for review February 3, 2024 01:32
@barrieloydall barrieloydall force-pushed the barrie/resolve-mix-docs-issue branch from 33b7f9f to 0573da7 Compare February 3, 2024 08:45
@barrieloydall barrieloydall changed the title resolve mix docs issue resolve mix docs issue + dependency updates Feb 3, 2024
@barrieloydall barrieloydall requested review from a team and removed request for ryanatwork February 3, 2024 20:23
@barrieloydall barrieloydall merged commit af336fc into master Feb 6, 2024
6 checks passed
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.

2 participants