Skip to content
This repository has been archived by the owner on Oct 3, 2024. It is now read-only.

Add linting check to ensure an email address is provided in the maintainer field #20

Closed
shonfeder opened this issue Aug 7, 2024 · 2 comments · Fixed by #30
Closed

Add linting check to ensure an email address is provided in the maintainer field #20

shonfeder opened this issue Aug 7, 2024 · 2 comments · Fixed by #30
Assignees
@punchagan

Comments

@shonfeder
Copy link
Collaborator

To address ocaml/infrastructure#152

I propose we do not fuss with the opam-repo-ci code, expecting it to be replaced with the logic here shortly.
@shonfeder shonfeder self-assigned this Aug 8, 2024
@punchagan punchagan mentioned this issue Aug 28, 2024
Merged
@punchagan punchagan assigned punchagan and unassigned shonfeder Aug 28, 2024
@punchagan
Copy link
Collaborator

@shonfeder apologies for not noticing that you had this issue assigned to yourself, and sending a PR for this.

@shonfeder
Copy link
Collaborator Author

Now worries at all. I probably shouldn't have assigned it to myself!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@punchagan punchagan

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Check for maintainer emails in the opam file punchagan/opam-ci-check
2 participants
@punchagan @shonfeder