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

[c/d-de] fix whitespace see discussion in PR #3839 #3860

Closed
wants to merge 3 commits into from

Conversation

caminsha
Copy link
Contributor

@caminsha caminsha commented Feb 16, 2020

  • I solemnly swear that this is all original content of which I am the original author
  • Pull request title is prepended with [language/lang-code]
  • Pull request touches only one file (or a set of logically related files with similar changes made)
  • Content changes are aimed at intermediate to experienced programmers (this is a poor format for explaining fundamental programming concepts)
  • If you've changed any part of the YAML Frontmatter, make sure it is formatted according to CONTRIBUTING.md
    • Yes, I have double-checked quotes and field names!

I changed the indentation because of a discussion in PR adambard#3839.

I'm thinking about the other kernel conventions because I think some are not very usefull.
@caminsha caminsha changed the title [De de/c] fix whitespace see discussion in PR #3839 [c/d-de] fix whitespace see discussion in PR #3839 Feb 16, 2020
@caminsha
Copy link
Contributor Author

I'm sorry, there just too many files.

I delete this PR and create a new one.

@caminsha caminsha closed this Feb 16, 2020
@caminsha caminsha deleted the de-de/c/fix_whitespace branch February 16, 2020 21:27
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

1 participant