Skip to content

Update IIS readme #3122

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
Merged
ellis-elastic merged 4 commits into from
Jun 16, 2022
Merged

Update IIS readme #3122

ellis-elastic merged 4 commits into from
Jun 16, 2022

Conversation

@ellis-elastic
Copy link
Contributor

@ellis-elastic ellis-elastic commented Apr 19, 2022 •
edited
Loading

updated readme to include link to IIS documentation.

Type of change

  • Enhancement

What does this PR do?

updated readme to include link to IIS documentation.

Checklist

  • I have reviewed tips for building integrations and this pull request is aligned with them.
  • I have verified that all data streams collect metrics or logs.
  • I have added an entry to my package's changelog.yml file.
  • I have verified that Kibana version constraints are current according to guidelines.

Author's Checklist

  • [ ]

How to test this PR locally

Related issues

Screenshots

@ellis-elastic
Update readme
af93e54 
updated readme to include link to IIS documentation.
@ellis-elastic ellis-elastic requested a review from a team as a code owner April 19, 2022 15:01
@ellis-elastic
Update changelog and ran build
007e6b6
@ellis-elastic
update change log
8cfcf2b 
added correct description to changelog
@ellis-elastic ellis-elastic added the documentation Improvements or additions to documentation. Applied to PRs that modify *.md files. label Apr 19, 2022
@ellis-elastic ellis-elastic marked this pull request as draft April 19, 2022 15:10
@elasticmachine
Copy link

elasticmachine commented Apr 19, 2022 •
edited
Loading

💚 Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS
Pipeline View Test View Changes Artifacts preview preview

Expand to view the summary

Build stats

  • Start Time: 2022-06-16T08:15:41.440+0000

  • Duration: 12 min 30 sec

Test stats 🧪

Test Results
Failed 0
Passed 84
Skipped 0
Total 84

🤖 GitHub comments

To re-run your PR in the CI, just comment with:

  • /test : Re-trigger the build.

@ellis-elastic ellis-elastic marked this pull request as ready for review April 20, 2022 11:36
@ellis-elastic ellis-elastic added the enhancement New feature or request label Apr 20, 2022
@elasticmachine
Copy link

elasticmachine commented May 10, 2022 •
edited
Loading

🌐 Coverage report

Name Metrics % (covered/total) Diff
Packages 100.0% (2/2) 💚
Files 100.0% (2/2) 💚 3.546
Classes 100.0% (2/2) 💚 3.546
Methods 85.714% (30/35) 👎 -3.608
Lines 91.549% (260/284) 👍 0.692
Conditionals 100.0% (0/0) 💚

@ruflin ruflin added the Team:Service-Integrations Label for the Observability Service Integrations team label Jun 8, 2022
@ellis-elastic ellis-elastic requested review from a team and removed request for a team June 16, 2022 06:27
@muthu-mps muthu-mps self-requested a review June 16, 2022 06:32
muthu-mps
muthu-mps approved these changes Jun 16, 2022
View reviewed changes
Copy link
Contributor

@muthu-mps muthu-mps left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please resolve the conflicts. Otherwise LGTM!

@ellis-elastic ellis-elastic merged commit aed6e1c into elastic:main Jun 16, 2022
@ellis-elastic ellis-elastic deleted the iis-docs branch June 16, 2022 09:52
@muthu-mps muthu-mps mentioned this pull request Jun 28, 2022
Closed
16 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@bmorelli25 bmorelli25 bmorelli25 approved these changes

@muthu-mps muthu-mps muthu-mps approved these changes

Assignees

No one assigned

Labels

documentation Improvements or additions to documentation. Applied to PRs that modify *.md files. enhancement New feature or request Team:Service-Integrations Label for the Observability Service Integrations team

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@ellis-elastic @elasticmachine @bmorelli25 @muthu-mps @ruflin