-
Notifications
You must be signed in to change notification settings - Fork 199
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
Add TinkerMail by TinkerHost Service #560
Conversation
Linter OK:
|
Ready for merge |
Something is wrong with linter check. It did not notice file name is missing
|
Forgetting the JSON extension is a bit embarrassing, let me fix that right now |
Fixed @kerolasa |
I think there is an issue with the github action as it selects only json files so any other file won't hit the linter at all
I think here we'd need to either let all files to run through linter, even if these are not related (README) or to have some smarter solution with exclusion lists... |
@greenreader9 the template looks good to me. Meanwhile, I think the following change to github actions might work. Should we try @pawel-kow?
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
Adds Domain Connect support to upcoming TinkerMail service