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

First pass at fixing FXCop security issues #2

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jameshed
Copy link

Signed-off-by: James Hedrick jameshed@microsoft.com

Fixed a number of issues found in Microsoft FXCop (https://msdn.microsoft.com/en-us/library/bb429476.aspx)

Signed-off-by: James Hedrick <jameshed@microsoft.com>
@fwsmaster
Copy link
Contributor

Hi James,

That's an impressive list of changes. Thanks for submitting and all the work. I'll check and verify to get them merged as soon as possible.

Kind regards
SP Coordinator

@rubberduck203
Copy link

@jameshed would you be interesting in maintaining a fork of this project? It seems that issues & PRs are going unanswered.

@fwsmaster
Copy link
Contributor

Hello,

Of course anyone is free to setup and maintain forks of their own. The official SP main branch however is still maintained through the SP project. Several changes are on their way (v. 2.4) but it may still take a while to get them published.

Reactions may sometimes be slow (due to a lack of assistance or available time/ capacity). In case of the changes as submitted by James they are under review.

Not all of them are acceptable which causes a lot of handwork (Github doesn support partial rejections).

Other issues in the issue list can be worked on by anyone interested to contribute. Any assistance otherwise is most welcome, just let us know.

Any urgent requests/ questions can be raised through the forums of the Surveproject.org website as well.

Kind regards
SP coordinator

@fwsmaster fwsmaster added this to the SP v. 2.5 Release milestone Jul 1, 2016
@w3devpro w3devpro removed this from the SP v. 2.5 Release milestone Mar 6, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants