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

Add D3 Visualization for test Results #141

Open
wants to merge 11 commits into
base: master
Choose a base branch
from
Open

Add D3 Visualization for test Results #141

wants to merge 11 commits into from

Conversation

bsherwin
Copy link
Contributor

No description provided.

bsherwin and others added 6 commits August 22, 2019 15:53
Includes the following changes:
1. Adds `metadata.json` result from the `compare` command as an attachment to the build pipeline.
2. Publishes `statistics.json` from the `compare` command as a build artifact to a container called `statistics` for `master` branch builds.
3. When `statistics.json` is published from a `master` branch build, adds a `nlu.devops.statistics` tag to the build so it can be queried for later.
4. Queries for the previous N (specified by user) builds with the `nlu.devops.statistics` tag, downloads their artifact archive and unzips.
5. Merges the `statistics.json` artifact from previous builds with the `statistics.json` for the current build.
6. Adds the merged `statistics.json` as an attachment to the build pipeline.
7. Adds a very basic Azure Pipelines web extension that looks for `metadata` and `statistics` attachments from the current pipeline and console.log's the link to those attachments.
extensions/tabs/matrix.css Outdated Show resolved Hide resolved
@@ -1,5 +1,5 @@
# Default
[{.,common,tasks}/**.{ts,json}]
[{.,common,tasks,tabs}/**.{ts,json,html,css}]
Copy link
Contributor

Choose a reason for hiding this comment

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

probably need to add .js to the file extensions checked.

// eslint-disable-next-line no-unused-vars
class Matrix {
constructor () {
this.p = 0
Copy link
Contributor

Choose a reason for hiding this comment

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

Looks like you can remove this.p and this.n, maybe that will allow you to remove eslint.

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