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.

Sign up for GitHub

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

Jump to bottom

fix: missing slz fsi input #63

Merged
merged 2 commits into from
Jan 7, 2025
Merged

fix: missing slz fsi input #63

merged 2 commits into from
Jan 7, 2025

Conversation

jaredfholgate
Copy link
Member

@jaredfholgate jaredfholgate commented Jan 7, 2025 •
edited
Loading

Overview/Summary

Add some checks in case config is missing

This PR fixes/adds/changes/removes

  1. Bug: Unsupported attribute - default_prefix, default_postfix & top_level_management_group_name ALZ-PowerShell-Module#267

Breaking Changes

None

Testing Evidence

Please provide any testing evidence to show that your Pull Request works/fixes as described and planned (include screenshots, if appropriate).

As part of this Pull Request I have

  • Checked for duplicate Pull Requests
  • Associated it with relevant issues, for tracking and closure.
  • Ensured my code/branch is up-to-date with the latest changes in the main branch
  • Performed testing and provided evidence.
  • Updated relevant and associated documentation.

@jaredfholgate jaredfholgate mentioned this pull request Jan 7, 2025
Closed
1 task
@jaredfholgate jaredfholgate added the PR: Safe to test 🧪 Enables running of End to End Tests label Jan 7, 2025
@jaredfholgate jaredfholgate self-assigned this Jan 7, 2025
@jaredfholgate jaredfholgate merged commit 34121c3 into main Jan 7, 2025
39 checks passed
@jaredfholgate jaredfholgate deleted the fix-missing-slz-fsi-input branch January 7, 2025 18:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@matt-FFFFFF matt-FFFFFF matt-FFFFFF approved these changes

@jtracey93 jtracey93 Awaiting requested review from jtracey93

Assignees

@jaredfholgate jaredfholgate

Labels
PR: Safe to test 🧪 Enables running of End to End Tests
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jaredfholgate @matt-FFFFFF