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

BCDA 7581: Update import env files message to support ticket completion #154

Merged
merged 2 commits into from
Jan 25, 2024

Conversation

alex-dzeda
Copy link
Contributor

🎫 Ticket

https://jira.cms.gov/browse/BCDA-7581

🛠 Changes

Improve utility of panic when trying to import env variables.

ℹ️ Context for reviewers

Current panic message is unhelpful, this adds debug info.

✅ Acceptance Validation

Change passes CI check

🔒 Security Implications

  • This PR adds a new software dependency or dependencies.
  • This PR modifies or invalidates one or more of our security controls.
  • This PR stores or transmits data that was not stored or transmitted before.
  • This PR requires additional review of its security implications for other reasons.

If any security implications apply, add Jason Ashbaugh (GitHub username: StewGoin) as a reviewer and do not merge this PR without his approval.

@alex-dzeda alex-dzeda merged commit 10f1dd0 into master Jan 25, 2024
3 checks passed
@alex-dzeda alex-dzeda deleted the alex/env_import_message branch January 25, 2024 19:10
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