Add sentry #653
Add sentry #653
Conversation
|
📦 To trigger a firmware build, comment
You can also specify a custom version: If no version is specified, one will be generated automatically. |
There was a problem hiding this comment.
Pull Request Overview
This PR introduces Sentry integration by packaging source code for Sentry in two GitHub Actions workflows. The changes include updating the release workflow to run Sentry packaging alongside building the release and adding a dedicated Sentry packaging step and environment variable (SENTRY_DSN) in the nerves workflow.
Reviewed Changes
Copilot reviewed 5 out of 18 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| .github/workflows/release.yml | Updated release command to include Sentry packaging |
| .github/workflows/nerves.yml | Added SENTRY_DSN environment variable and a new step for packaging source code for Sentry |
Files not reviewed (13)
- .github/workflows/Dockerfile: Language not supported
- .github/workflows/Dockerfile-arm: Language not supported
- Dockerfile: Language not supported
- nerves_fw/config/target.exs: Language not supported
- nerves_fw/lib/nerves_fw/hardware/rut.ex: Language not supported
- ui/config/dev.exs: Language not supported
- ui/config/prod.exs: Language not supported
- ui/lib/ex_nvr/sentry.ex: Language not supported
- ui/lib/ex_nvr_web.ex: Language not supported
- ui/lib/ex_nvr_web/application.ex: Language not supported
- ui/lib/ex_nvr_web/endpoint.ex: Language not supported
- ui/lib/ex_nvr_web/live/user_list_live.ex: Language not supported
- ui/mix.exs: Language not supported
Comments suppressed due to low confidence (2)
.github/workflows/release.yml:34
- Ensure that combining 'sentry.package_source_code' with 'release' in a single mix command does not cause unintended side effects in the release process. Consider testing that both operations execute in the correct order, or separate them if any issues arise.
run: mix do sentry.package_source_code, release
.github/workflows/nerves.yml:111
- Confirm that running 'mix sentry.package_source_code' as a standalone step after dependency installation correctly integrates with the firmware build process. Consistency in the Sentry integration approach across workflows should be verified.
- name: Package source code for sentry
No description provided.