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

fix: fix a type mistake #377

Closed
wants to merge 1 commit into from
Closed

Conversation

kkzhsh
Copy link
Contributor

@kkzhsh kkzhsh commented Dec 15, 2022

a type mistake?

Pls help to review it

@bachradsusi
Copy link
Member

Hello, thanks for the patch.

The contributing process is described in https://github.com/SELinuxProject/selinux/blob/master/CONTRIBUTING.md

  • it's important to Signed-off-by: line in the commit message with a real name and existing email address. You can use git commit -s --amend to update the commit.
  • please take look at other commit messages. In this case, 'fix' is there twice, I'd use something like: "sepolicy: fix a spelling mistake"
  • in order to get the patch reviewed and accepted, it's necessary to send it to [email protected] mailing list. The best way is to use git send-email command, e.g.:
    git send-email --from='Your Name <[email protected]>' [email protected] --smtp-server=my.smtp.server --confirm=auto -1

@kkzhsh
Copy link
Contributor Author

kkzhsh commented Dec 16, 2022

Dear @bachradsusi ,
I have sent a mail according to your comments.
Thank you

@perfinion
Copy link
Member

Merged in 60a0d7285d90680d0ce839ba6b169a33c91c5fb0
Thanks!

@perfinion perfinion closed this Jan 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants