-
Notifications
You must be signed in to change notification settings - Fork 39
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
Update/fancy tab #13
Update/fancy tab #13
Conversation
@check-spelling-bot Report🔴 Please reviewUnrecognized words (165)
Previously acknowledged words that are now absentEmoji Linux MULTILINE UintAvailable dictionaries could cover words not in the dictionarycspell:cpp/cpp.txt (104293) covers 150 of them Consider adding them using:
To stop checking additional dictionaries, add:
To accept these unrecognized words as correct (and remove the previously acknowledged and now absent words), run the following commands... in a clone of the git@github.com:LeStarch/fprime-gds.git repository
|
This pull request introduces 4 alerts when merging 01655cc into 0a374a4 - view on LGTM.com new alerts:
|
@check-spelling-bot Report🔴 Please reviewUnrecognized words (168)
Previously acknowledged words that are now absentEmoji Linux MULTILINE UintAvailable dictionaries could cover words not in the dictionarycspell:cpp/cpp.txt (104293) covers 151 of them Consider adding them using:
To stop checking additional dictionaries, add:
To accept these unrecognized words as correct (and remove the previously acknowledged and now absent words), run the following commands... in a clone of the git@github.com:LeStarch/fprime-gds.git repository
|
This pull request introduces 3 alerts when merging 708a0f1 into 0a374a4 - view on LGTM.com new alerts:
|
@check-spelling-bot Report🔴 Please reviewUnrecognized words (168)
Previously acknowledged words that are now absentEmoji Linux MULTILINE UintAvailable dictionaries could cover words not in the dictionarycspell:cpp/cpp.txt (104293) covers 151 of them Consider adding them using:
To stop checking additional dictionaries, add:
To accept these unrecognized words as correct (and remove the previously acknowledged and now absent words), run the following commands... in a clone of the git@github.com:LeStarch/fprime-gds.git repository
|
This pull request introduces 3 alerts when merging 2b55277 into 0a374a4 - view on LGTM.com new alerts:
|
@check-spelling-bot Report🔴 Please reviewUnrecognized words (14)
Available dictionaries could cover words not in the dictionarycspell:cpp/cpp.txt (104293) covers 128 of them Consider adding them using:
To stop checking additional dictionaries, add:
To accept these unrecognized words as correct, run the following commands... in a clone of the git@github.com:LeStarch/fprime-gds.git repository
|
191af93
to
b68fd58
Compare
This pull request introduces 3 alerts when merging b68fd58 into 0a374a4 - view on LGTM.com new alerts:
|
Python alert is not new, just tabbed in one. |
this is so cool! |
This pull request introduces 2 alerts when merging 536af64 into 0a374a4 - view on LGTM.com new alerts:
|
@LeStarch I believe you said you were going to fix these, given I'm running an older version of python3: |
@metzdigital should work on your revision now. |
This pull request introduces 2 alerts when merging 0181878 into 0a374a4 - view on LGTM.com new alerts:
|
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.
Approved.
This pull request introduces 2 alerts when merging 155b861 into 4e54110 - view on LGTM.com new alerts:
|
This pull request introduces 2 alerts when merging 1c33f6b into 4e54110 - view on LGTM.com new alerts:
|
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.
Approved.
Merging this change. Lint warnings are not new (just moved slightly) and |
This pull request introduces 2 alerts when merging b46b626 into 9994195 - view on LGTM.com new alerts:
|
Change Description
A description of the changes contained in the PR.
Rationale
A rationale for this change. e.g. fixes bug, or most projects need XYZ feature.
Testing/Review Recommendations
Fill in testing procedures, specific items to focus on for review, or other info to help the team verify these changes are flight-quality.
Future Work
Note any additional work that will be done relating to this issue.