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

new versions of Malcode Analyst Pack & scdbg #601

Closed
wants to merge 1 commit into from
Closed

new versions of Malcode Analyst Pack & scdbg #601

wants to merge 1 commit into from

Conversation

dzzie
Copy link

@dzzie dzzie commented Jul 27, 2023

No description provided.

@google-cla
Copy link

google-cla bot commented Jul 27, 2023

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

@dzzie
Copy link
Author

dzzie commented Aug 3, 2023

I am not going to sign the google CLA thing, because I cant be sure what email address they will attach to it and they have access to a private one.

Anyway the old git repos will go away in a week or so as I move completely off the platform. They are only up right now so I dont break your install scripts

@mr-tz
Copy link
Contributor

mr-tz commented Aug 7, 2023

thanks, for the heads-up and new links!

@mr-tz mr-tz mentioned this pull request Aug 7, 2023
@mr-tz
Copy link
Contributor

mr-tz commented Aug 8, 2023

is it ok if we use #621?

@dzzie
Copy link
Author

dzzie commented Aug 8, 2023

I just saw one of the auto checks was blocking because version was not updated in the nuspec's so added that as well

@Ana06
Copy link
Member

Ana06 commented Sep 7, 2023

@dzzie sorry for the late reply and thank you so much for keeping the old version to avoid breaking our package.

I am very sorry that the signing of the CLA discourages you to contribute to VM-Packages. I could ask about the email attached to the CLA (or any other questions you have) if this would make it any easier.

Without the CLA being signed, we can't merge this PR. So I am closing this PR in favour of #621, there is a small problem with the version: https://github.com/mandiant/VM-Packages/pull/621/files#r1318383221 After we have fixed that, we can get it merged.

@Ana06 Ana06 closed this Sep 7, 2023
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.

3 participants