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.

Sign up for GitHub

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

Jump to bottom

[improve][misc] Add security section to PIP issue template #19803

Merged
merged 1 commit into from
Mar 14, 2023
Merged

[improve][misc] Add security section to PIP issue template #19803

merged 1 commit into from
Mar 14, 2023

Conversation

michaeljmarshall
Copy link
Member

Motivation

PIPs often add new features, and it is always important to consider security during the design and discussion phase of a PIP.

Modifications

  • Add security section to PIP issue template

Documentation

  • doc-not-needed

@michaeljmarshall michaeljmarshall added doc-not-needed Your PR changes do not impact docs ready-to-test labels Mar 14, 2023
@michaeljmarshall michaeljmarshall self-assigned this Mar 14, 2023
eolivelli
eolivelli approved these changes Mar 14, 2023
View reviewed changes
Copy link
Contributor

@eolivelli eolivelli left a comment

Choose a reason for hiding this comment

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

+1

@tisonkun tisonkun changed the title [improve][repo] Add security section to PIP issue template [improve][misc] Add security section to PIP issue template Mar 14, 2023
@nodece nodece merged commit 4f6b5fe into apache:master Mar 14, 2023
@michaeljmarshall michaeljmarshall deleted the add-security-section-to-pip branch March 14, 2023 22:45
@michaeljmarshall michaeljmarshall mentioned this pull request Mar 16, 2023
Closed
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tisonkun tisonkun tisonkun approved these changes

@codelipenghui codelipenghui codelipenghui approved these changes

@eolivelli eolivelli eolivelli approved these changes

@merlimat merlimat Awaiting requested review from merlimat

@lhotari lhotari Awaiting requested review from lhotari

@nodece nodece Awaiting requested review from nodece

@nicoloboschi nicoloboschi Awaiting requested review from nicoloboschi

@dave2wave dave2wave Awaiting requested review from dave2wave

Assignees

@michaeljmarshall michaeljmarshall

Labels
doc-not-needed Your PR changes do not impact docs ready-to-test
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@michaeljmarshall @eolivelli @codelipenghui @tisonkun @nodece