-
Notifications
You must be signed in to change notification settings - Fork 205
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
blackduck in azure pipeline #461
Conversation
649acef
to
0e7ea3d
Compare
Can we close this PR or are you still planning to work on this? |
Moritz
Still work to do on this one
On Mon, Sep 16, 2019 at 05:52 Moritz Kiefer ***@***.***> wrote:
Can we close this PR or are you still planning to work on this?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#461?email_source=notifications&email_token=AGADHCJYJBH6QFDUJQMTSETQJ5JPHA5CNFSM4HFSN2D2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD6YUIBA#issuecomment-531710980>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AGADHCNX2ERTZ2ANVCGATCLQJ5JPHANCNFSM4HFSN2DQ>
.
--
Brian Healey
Senior Software Engineer
c: +1 646 932 8942
e: brian.healey@digitalasset.com
*Digital Asset* <https://digitalasset.com/>, creators of *DAML
<https://daml.com/>*
…--
This message, and any attachments, is for the intended recipient(s) only,
may contain information that is privileged, confidential and/or proprietary
and subject to important terms and conditions available at
http://www.digitalasset.com/emaildisclaimer.html
<http://www.digitalasset.com/emaildisclaimer.html>. If you are not the
intended recipient, please delete this message.
|
@dasormeter Can you clarify what need to be done on this? It looks stale to me. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please delete code instead of commenting it out. git
will do a good job at preserving this for the future if we want to restore it.
Yeah just draft pr for now - will tidy up once proven working on CI
On Sat, Apr 18, 2020 at 04:47 Stefano Baghino ***@***.***> wrote:
***@***.**** commented on this pull request.
Please delete code instead of commenting it out. git will do a good job
at preserving this for the future if we want to restore it.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#461 (review)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AGADHCKQ6EIWTTEHQH3YAXTRNFSJVANCNFSM4HFSN2DQ>
.
--
Brian Healey
Senior Software Engineer
c: +1 646 932 8942
e: brian.healey@digitalasset.com
*Digital Asset* <https://digitalasset.com/>, creators of *DAML
<https://daml.com/>*
…--
This message, and any attachments, is for the intended recipient(s) only,
may contain information that is privileged, confidential and/or proprietary
and subject to important terms and conditions available at
http://www.digitalasset.com/emaildisclaimer.html
<http://www.digitalasset.com/emaildisclaimer.html>. If you are not the
intended recipient, please delete this message.
|
superceded by #6130 |
Pull Request Checklist
Enable Blackduck scan to be run on master only for DAML repo