Skip to content
This repository has been archived by the owner on Jun 12, 2023. It is now read-only.

Accred test #325

Merged
merged 3 commits into from
Jan 7, 2020
Merged

Accred test #325

merged 3 commits into from
Jan 7, 2020

Conversation

dcmckayibm
Copy link
Collaborator

Summary

Details and comments

sethmerkel and others added 3 commits December 13, 2019 13:45
@sethmerkel
added unittest for accred
26412c6 
Added unit tests for accreditation (following the tests for QV)
@dcmckayibm
Merge branch 'master' into master
830b5f4
@dcmckayibm
Fixed lint issues
e13a120
@claassistantio
Copy link

claassistantio commented Jan 7, 2020 •
edited
Loading

CLA assistant check
All committers have signed the CLA.

@claassistantio
Copy link

CLA assistant check
Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 2 committers have signed the CLA.

✅ dcmckayibm
❌ sethmerkel
You have signed the CLA already but the status is still pending? Let us recheck it.

sethmerkel
sethmerkel reviewed Jan 7, 2020
View reviewed changes
Copy link
Contributor

@sethmerkel sethmerkel left a comment

Choose a reason for hiding this comment

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

looks good still

@dcmckayibm dcmckayibm merged commit 559d5b2 into qiskit-community:master Jan 7, 2020
@dcmckayibm dcmckayibm deleted the accred_test branch January 7, 2020 17:33
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@sethmerkel sethmerkel sethmerkel left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@dcmckayibm @claassistantio @sethmerkel