Skip to content

Commit

Permalink
docs: update (#71)
Browse files Browse the repository at this point in the history
* docs: update contributing

* fix: add sonarcloud + update schema

* fix
  • Loading branch information
Julien Bouquillon authored Apr 19, 2023
1 parent a64ceaf commit 068ebcc
Show file tree
Hide file tree
Showing 5 changed files with 207 additions and 138 deletions.
24 changes: 8 additions & 16 deletions .github/workflows/report.yml
Original file line number Diff line number Diff line change
Expand Up @@ -3,10 +3,10 @@ name: DashLord report
on:
workflow_dispatch:
workflow_run:
workflows: ["DashLord scans"]
branches: [main]
types:
- completed
workflows: ["DashLord scans"]
branches: [main]
types:
- completed

# allow only one concurrent report action
concurrency:
Expand All @@ -24,27 +24,20 @@ jobs:

- uses: actions/cache@v2
with:
path: '**/node_modules'
path: "**/node_modules"
key: ${{ runner.os }}-modules-${{ hashFiles('**/yarn.lock') }}

# build the report
- id: dashlord-report
uses: SocialGouv/dashlord-actions/report@main
uses: SocialGouv/dashlord-actions/report@v1
with:
base-path: /dashlord

# to save the generated report.json as artifact
#- uses: actions/upload-artifact@v2
# with:
# path: report.json
# name: report
# if-no-files-found: error
base-path: /dashlord # adapt to your repo name

# save full report for history
- uses: EndBug/add-and-commit@v7
with:
add: '["report.json"]'
default_author: github_actions
default_author: "github_actions"
message: "chore: report update"

# deploy build to gh-pages
Expand All @@ -53,4 +46,3 @@ jobs:
with:
branch: gh-pages
folder: build

Loading

2 comments on commit 068ebcc

@arthurzenika
Copy link
Contributor

Choose a reason for hiding this comment

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

@revolunet : is it intentional to remove the nmap scan with this commit ? If so maybe the default configuration should remove the attribute and the html report remove the column ? Re-adding it to a project created recently to see if it still works.

@revolunet
Copy link
Member

Choose a reason for hiding this comment

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

no its not, thanks for noticing

Please sign in to comment.