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

FR Header Checks: Update expected file header. Support ignores. #410

Merged
merged 2 commits into from
Nov 18, 2020

Conversation

dachalco
Copy link
Contributor

@dachalco dachalco commented Nov 18, 2020

  • Remove the * 1 tab == 4 spaces remark from header check.
  • Add ignore mechanism
  • Use ignore mechanism to skip checks for certain file extensions, filenames, including the checker script itself

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@dachalco dachalco requested a review from a team as a code owner November 18, 2020 16:54
cobusve
cobusve previously approved these changes Nov 18, 2020
@dachalco dachalco force-pushed the check/file-header-plus-gitaction branch from c6b8a64 to 31de662 Compare November 18, 2020 20:11
@dachalco
Copy link
Contributor Author

Added another update in last commit. Introduces ignore mechanism, and uses it to ignore some files. Likely this ignore list will grow

@dachalco dachalco force-pushed the check/file-header-plus-gitaction branch from 31de662 to 60c4a6b Compare November 18, 2020 20:17
@dachalco dachalco changed the title Remove '1 tab == 4 spaces remarks' from check FR Header Checks: Update expected file header. Support ignores. Nov 18, 2020
@dachalco dachalco requested a review from cobusve November 18, 2020 20:44
@dachalco dachalco merged commit 0d0a888 into FreeRTOS:master Nov 18, 2020
moninom1 pushed a commit that referenced this pull request Sep 14, 2022
* Always declare SignalSocket functions regardless of TCP

* Fix socket header declarations based on ipconfig values

* Socket header file cleanup and formatting

* Uncrustify: triggered by comment.

* Address reformatting mistakes and add documentation

* Uncrustified

* Fix missing constants

* Fix build errors

* Add to lexicon and re-add inclusion checks

* Add missing spelling check

* Convert xTCP_UDP_HANDLER from a union to a struct

Co-authored-by: GitHub Action <[email protected]>
Co-authored-by: Aniruddha Kanhere <[email protected]>
paulbartell pushed a commit to paulbartell/FreeRTOS that referenced this pull request Mar 30, 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

2 participants