Skip to content

Add maintainers field #408

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
Merged
merged 1 commit into from
Sep 22, 2020
Merged

Add maintainers field #408

merged 1 commit into from
Sep 22, 2020

Conversation

jpreese
Copy link
Member

@jpreese jpreese commented Sep 22, 2020

Resolves #399

This field appears to be required by dpkg and .. does not support multiple aliases?

@jpreese
Add maintainers field
6bdab3b 
Signed-off-by: John Reese <john@reese.dev>
@jpreese jpreese requested a review from Blokje5 September 22, 2020 14:26
@jpreese jpreese mentioned this pull request Sep 22, 2020
Closed
Blokje5
Blokje5 approved these changes Sep 22, 2020
View reviewed changes
Copy link
Collaborator

@Blokje5 Blokje5 left a comment

Choose a reason for hiding this comment

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

LGTM! In the future we can instead of your email set up a shared conftest maintainers email address and integrate it with e.g. the OPA slack/Github. That way we can ensure we recieve e.g. Debian BTS reports.

@Blokje5 Blokje5 merged commit 67f5f4e into open-policy-agent:master Sep 22, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Blokje5 Blokje5 Blokje5 approved these changes

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

Successfully merging this pull request may close these issues.

dpkg-query: warning: parsing file '/var/lib/dpkg/status' .... missing 'Maintainer' field
2 participants
@jpreese @Blokje5