Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Converted Application Insights to bicep #114

Merged
merged 40 commits into from
Oct 5, 2021
Merged

Converted Application Insights to bicep #114

merged 40 commits into from
Oct 5, 2021

Conversation

MariusStorhaug
Copy link
Contributor

Change

  • Converted Application Insights (components) to bicep

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update (Wiki)

Checklist

  • I'm sure there are no other open Pull Requests for the same update/change
  • My corresponding pipelines / checks run clean and green without any errors or warnings
  • My code follows the style guidelines of this project
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation (readme)
  • I did format my code

@MariusStorhaug MariusStorhaug merged commit 5591935 into Azure:main Oct 5, 2021
@MariusStorhaug MariusStorhaug deleted the appInsights branch October 5, 2021 10:49
@github-actions
Copy link

github-actions bot commented Oct 5, 2021

Unit Test Results

  1 files  ±0  1 suites  ±0   8s ⏱️ ±0s
  2 tests ±0  2 ✔️ ±0    0 💤 ±0  0 ±0 
41 runs  ±0  3 ✔️ ±0  38 💤 ±0  0 ±0 

Results for commit 5591935. ± Comparison against base commit 5591935.

♻️ This comment has been updated with latest results.

@MariusStorhaug MariusStorhaug self-assigned this May 19, 2022
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants