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.

Sign up for GitHub

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

Jump to bottom

update documentation #6035

Merged
merged 6 commits into from
Dec 7, 2022
Merged

update documentation #6035

merged 6 commits into from
Dec 7, 2022

Conversation

gabriel-cx
Copy link
Contributor

Proposed Changes

  • Documentation update

I submit this contribution under the Apache-2.0 license.

@github-actions
Copy link
Contributor

github-actions bot commented Dec 5, 2022 •
edited
Loading

kics-logo

KICS version: v1.6.4

Category Results
HIGH HIGH 0
MEDIUM MEDIUM 0
LOW LOW 0
INFO INFO 0
TRACE TRACE 0
TOTAL TOTAL 0
Metric Values
Files scanned placeholder 1
Files parsed placeholder 1
Files failed to scan placeholder 0
Total executed queries placeholder 49
Queries failed to execute placeholder 0
Execution time placeholder 1

@gabriel-cx gabriel-cx requested a review from kaplanlior December 6, 2022 12:54
cxMiguelSilva
cxMiguelSilva reviewed Dec 7, 2022
View reviewed changes
Copy link
Collaborator

@cxMiguelSilva cxMiguelSilva left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

image
Maybe we could change the display order to make the getting started to appear first.
Let me know what you think

@gabriel-cx
Copy link
Contributor Author

image Maybe we could change the display order to make the getting started to appear first. Let me know what you think

@kaplanlior , what do you think about this propose?

cxMiguelSilva
cxMiguelSilva approved these changes Dec 7, 2022
View reviewed changes
Copy link
Collaborator

@cxMiguelSilva cxMiguelSilva left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@cxMiguelSilva cxMiguelSilva merged commit f3f47b5 into master Dec 7, 2022
@cxMiguelSilva cxMiguelSilva deleted the docrefactoring branch December 7, 2022 14:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@cxMiguelSilva cxMiguelSilva cxMiguelSilva approved these changes

@kaplanlior kaplanlior Awaiting requested review from kaplanlior

@cx-joao-reigota cx-joao-reigota Awaiting requested review from cx-joao-reigota

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@gabriel-cx @cxMiguelSilva