Skip to content

Update readme to include section on missing analysis - #599

Merged
aeisenberg merged 2 commits into
mainfrom
aeisenberg/readme-missing-analysis
Jun 28, 2021
Merged

Update readme to include section on missing analysis#599
aeisenberg merged 2 commits into
mainfrom
aeisenberg/readme-missing-analysis

Conversation

@aeisenberg

Copy link
Copy Markdown
Contributor

Fixes#598

Merge / deployment checklist

  • Confirm this change is backwards compatible with existing workflows.
  • Confirm the readme has been updated if necessary.
  • [n/a] Confirm the changelog has been updated if necessary.

@aeisenberg
aeisenberg requested a review from a team as a code ownerJune 25, 2021 17:28
@aeisenberg

Copy link
Copy Markdown
ContributorAuthor

The new text in the readme is mostly copied from the original docs page, which describes the issue so nicely I didn't want to make any changes.

@adityasharadadityasharad left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks! Minor suggestions.

Comment threadREADME.md Outdated
Comment threadREADME.md Outdated
@aeisenberg
aeisenbergforce-pushed the aeisenberg/readme-missing-analysis branch from 8422d26 to 7c391e9CompareJune 25, 2021 17:44
@aeisenberg
aeisenberg enabled auto-merge June 25, 2021 17:45
@aeisenberg
aeisenberg merged commit 3a8e184 into mainJun 28, 2021
@aeisenberg
aeisenberg deleted the aeisenberg/readme-missing-analysis branch June 28, 2021 15:30
@github-actionsgithub-actionsBot mentioned this pull request Jul 5, 2021
5 tasks
@github-actionsgithub-actionsBot mentioned this pull request Jul 12, 2021
5 tasks
Sign up for freeto 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.

Enable configuration to fail CI if new alerts are introduced

2 participants

@aeisenberg@adityasharad