* Adding gosec linter
Adding gosec (Static application security testing) linter.
* Testing gosec exclusion in golangci-lint
G110 is being triggered - testing if exclude works, then will revert and leave the excludes section there for future needs.
Also adding some tests to exclude rules
* Update .golangci.yml
Testing gosec allow list of rules
* Update .golangci.yml
Trying gosec deny list of rules
* Update .golangci.yml
Reconfiguring G306 alert to use 644, less stringent, and adding every file with current warnings to exclusions. Once this is all buttoned up and running on NEW files, we will go back and address all issues or put a comment stating why the file is excluded here.
* Update .golangci.yml
Removing exclusion to see if lint now fails
* Update .golangci.yml
Removing most exclusions to test
* Update .golangci.yml
Removing 100% of exclusions for more testing
* Update .golangci.yml
Re-adding exclusions slowly and testing to be sure we only exclude what is needed
* Update .golangci.yml
Adding all exclusions and comments per category pointing to github issues for investigating and cleaning up over time.
* Update community.md
Added to "Posting on social media as Fleet"
* Update community.md
Recorded by line:
34 deleted "to do so"
42 added "," after "drafted"
61 moved "easily"
63 reworded
81 deleted "that is"
86 replaced "in which" with "when"
102 replaced "large" with "significant"
107 replaced "a user" with "users"; added "user interface" for clarity; replaced ", but" with ". Still"
110 removed "()" from "Screenshots can get out of date quickly as UIs change."
113 removed "and beyond"; removed "that" after "make sure"
116 replaced "url" with "URL"
124 added "is" before "written"
126 corrected spelling for "specified"
154 deleted "," after "story"
158 deleted quotes around "think"
167 deleted "," after "(within 1 business day)"
174 replaced "Issue" with "issue"
177 replaced "get" with "make"; replaced "quality" with "content"
178 added "to" after "changes"
179 added "," after "Usually"
180 added "," after "customers)"
182-183 reworded to "After reviewing a PR and addressing all necessary changes any Fleet team member may merge a"
183 reworded to "community it."
184 corrected spelling of "judgment"
185 replaced "Folks" with "Those"
191 replaced "small" with "minor"; corrected "bug fix"; deleted "," after "acceptable"
192 added (SQL) for clarity
193 shifted "the" down; added "are" before "attributed"; corrected spelling of "judgment"
210 replaced "feel" with "Feel"
* Update community.md
Updating printful link
Co-authored-by: Desmi-Dizney <99777687+Desmi-Dizney@users.noreply.github.com>
* Handbook: incorporate feedback from team conversations in late 2021
* incorporate Ben/Martin/Noah/Tony feedback
* Update company.md
* reorganize some of the new ideas to be easier to read (#4485)
* more cleanup
* Update company.md
* Update company.md
* Allow sort by more than one key
* created custom tooltip component
* remove unused code
* fixed style for more layouts
* added tooltip to query side panel
* tooltips added to setting form
* finished settings form
* added tooltip for perf impact table headers
* tooltip for pack table and user form
* tooltip on manage policies page
* tooltip for manage schedules
* tooltip for automations; spacing for form input
* tooltip for automations modal
* user form; fixed input with icon component
* more user form tooltips
* tooltip for homepage; style fixes
* replaced many more tooltips with new version
* added story for tooltip
* added position prop
* fixed tests
* re-work how we click react-select dropdowns
* forcing the update button click
* trying a blur
* fixed typo
* trying blur on another element
* temp check-in
* replaced tooltip from host details software
* more consolidation of tooltip use for software
* fixed settings flow test
Co-authored-by: Tomas Touceda <chiiph@gmail.com>
* mike-t-handbook-patch
• Added Fleet swag process to handbook
* Update community.md
Recorded by line:
3 SUGGESTION: Was "open-core" meant to be "open-source"?
49 Reworded sentence for clarity.
51 Added quotes to "-" and "_".
Co-authored-by: Desmi-Dizney <99777687+Desmi-Dizney@users.noreply.github.com>
* Error early if email already exists
* Update changes/issue-4361-mail-change-should-error
Co-authored-by: Michal Nicpon <39177923+michalnicp@users.noreply.github.com>
* Fix test
* Lint fixes
* Fix another test
* Also check for invites
* Improve error checks
* Update comment
* Update tests
* Fix test
Co-authored-by: Michal Nicpon <39177923+michalnicp@users.noreply.github.com>
gillespi314