-
Notifications
You must be signed in to change notification settings - Fork 130
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
chore(ci): bump super-linter/super-linter from 6 to 7 in /.github/workflows #602
chore(ci): bump super-linter/super-linter from 6 to 7 in /.github/workflows #602
Conversation
Bumps [super-linter/super-linter](https://github.com/super-linter/super-linter) from 6 to 7. - [Release notes](https://github.com/super-linter/super-linter/releases) - [Changelog](https://github.com/super-linter/super-linter/blob/main/CHANGELOG.md) - [Commits](super-linter/super-linter@v6...v7) --- updated-dependencies: - dependency-name: super-linter/super-linter dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <[email protected]>
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
WalkthroughThe primary changes involve upgrading the Super Linter in the GitHub Actions workflow from Changes
Poem
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (invoked as PR comments)
Additionally, you can add CodeRabbit Configuration File (
|
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.
Yeah! You did it 🎉 Now, Relax 😉, Grab a drink ☕, and wait for the maintainers to check your contributions. Meanwhile, you can discuss on other issues and solve them 😀. Thank You 😃!
Meanwhile you can also discuss about the project in our Discord Server 😀
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.
Actionable comments posted: 0
Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files selected for processing (1)
- .github/workflows/linter.yml (1 hunks)
Files skipped from review due to trivial changes (1)
- .github/workflows/linter.yml
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.
Actionable comments posted: 1
Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files selected for processing (4)
- .github/workflows/formatter.yml (2 hunks)
- .github/workflows/linter.yml (1 hunks)
- .prettierignore (1 hunks)
- README.md (5 hunks)
Files skipped from review due to trivial changes (1)
- .prettierignore
Files skipped from review as they are similar to previous changes (1)
- .github/workflows/linter.yml
Additional context used
LanguageTool
README.md
[typographical] ~171-~171: Do not use a colon (:) before a series that is introduced by a preposition (‘below’). Remove the colon or add a noun or a noun phrase after the preposition.
Context: ...Run Anyway as present on the screen below: ||| |--|--| | ![Windows Def...(RP_COLON)
Additional comments not posted (5)
.github/workflows/formatter.yml (3)
14-17
: Job renaming and permission changes are appropriate.The job name change reflects the broader scope of file formatting, and the permission adjustment to
contents: write
is a good security practice.
30-33
: Prettier installation and usage steps are well-implemented.Installing Prettier globally and using it to format CSS, JavaScript, YAML, JSON, and Markdown files aligns with the workflow's expanded scope.
40-40
: Commit message update is appropriate.The updated commit message "style: format codebase" accurately reflects the broader scope of file formatting.
README.md (2)
2-8
: Stylistic updates enhance readability and compliance.The use of
style="text-align: center;"
complies with modern HTML standards, and the minor text edits improve consistency.Also applies to: 13-18
26-27
: Content adjustments improve clarity and consistency.The minor changes in phrasing and capitalization enhance the document's readability and presentation.
Also applies to: 29-55, 70-179
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.
Actionable comments posted: 0
Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files selected for processing (1)
- .github/workflows/formatter.yml (2 hunks)
Files skipped from review as they are similar to previous changes (1)
- .github/workflows/formatter.yml
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.
Actionable comments posted: 2
Outside diff range, codebase verification and nitpick comments (6)
Website/app/docs/Usage/drifty-gui/downloading-files.md (1)
19-19
: Avoid using bare URLs.The URL on line 19 is a bare URL. Consider formatting it as a hyperlink for consistency and improved readability.
- https://github.com/user-attachments/assets/48349e1a-f741-4f17-977d-91a1d6876b2a + ![Download Queue](https://github.com/user-attachments/assets/48349e1a-f741-4f17-977d-91a1d6876b2a)Tools
Markdownlint
19-19: null
Bare URL used(MD034, no-bare-urls)
Website/app/docs/Usage/drifty-cli/downloading-files.md (1)
16-19
: Consider varying the phrasing for improved readability.The repeated use of "if you want to" can be varied to enhance the document's readability. Consider using alternative expressions.
- - `.` if you want to download the video in the default download directory (which is `{user_home}/Downloads` in most cases). - - `L` if you want to use the last used directory. - - the absolute path of the directory where you want to save the video, e.g., `/home/user/Videos`. + - `.` to download the video in the default download directory (which is `{user_home}/Downloads` in most cases). + - `L` to use the last used directory. + - the absolute path of the directory for saving the video, e.g., `/home/user/Videos`.Tools
LanguageTool
[style] ~17-~17: You have already used this phrasing in nearby sentences. Consider replacing it to add variety to your writing.
Context: ...tination folder. Enter -.
if you want to download the video in the default downl...(REP_WANT_TO_VB)
[style] ~18-~18: You have already used this phrasing in nearby sentences. Consider replacing it to add variety to your writing.
Context: ...nloadsin most cases). -
L` if you want to use the last used directory. - the a...(REP_WANT_TO_VB)
[style] ~19-~19: You have already used this phrasing in nearby sentences. Consider replacing it to add variety to your writing.
Context: ...bsolute path of the directory where you want to save the video, e.g., `/home/user/Video...(REP_WANT_TO_VB)
CODE_OF_CONDUCT.md (1)
20-36
: Improved readability and streamlined phrasing.The removal of extra line breaks enhances document readability. Consider using a synonym for "insulting" to strengthen the wording.
- Trolling, insulting or derogatory comments, and personal or political attacks + Trolling, disparaging or derogatory comments, and personal or political attacksTools
LanguageTool
[style] ~32-~32: Try using a synonym here to strengthen your wording.
Context: ...ind - Trolling, insulting or derogatory comments, and personal or political attacks - Pu...(COMMENT_REMARK)
Website/app/docs/Development/macOS-docker-build-instructions.md (1)
83-83
: Add a comma for clarity.Consider adding a comma to improve the sentence structure.
- regardless of your ip address and your router will always be `.1` (in most cases) + regardless of your ip address, and your router will always be `.1` (in most cases)Tools
LanguageTool
[uncategorized] ~83-~83: Possible missing comma found.
Context: ...e I have it here, regardless of your ip address and your router will always be.1
(in...(AI_HYDRA_LEO_MISSING_COMMA)
Docker/macOS Docker Build Instructions.md (1)
32-32
: Consider rephrasing for variety.The phrase "If you are going to" is repeated in nearby sentences. Consider rephrasing to add variety to the writing.
- If you are going to use the pre-built **Drifty GUI** Docker image from [**GitHub Container registry releases**](https://ghcr.io/saptarshisarkar12/drifty-gui), then run this command: + To use the pre-built **Drifty GUI** Docker image from [**GitHub Container registry releases**](https://ghcr.io/saptarshisarkar12/drifty-gui), execute the following command:Tools
LanguageTool
[style] ~32-~32: You have already used this phrasing in nearby sentences. Consider replacing it to add variety to your writing.
Context: ...h docker compose build gui ``` - If you are going to use the pre-built Drifty GUI Docker...(REP_GOING_TO_VB)
README.md (1)
98-98
: Consider using a more expressive alternative.The phrase "To do this" can be replaced with a more expressive alternative for clarity.
- To do this, add **_`sudo`_** before the docker commands. + To achieve this, prepend **_`sudo`_** to the docker commands.Tools
LanguageTool
[style] ~98-~98: Consider a more expressive alternative.
Context: ...ning images with Docker Desktop. To do this, addsudo
before the docke...(DO_ACHIEVE)
Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files selected for processing (56)
- .github/ISSUE_TEMPLATE/Bug-for-application.yaml (1 hunks)
- .github/ISSUE_TEMPLATE/Bug-for-website.yaml (1 hunks)
- .github/ISSUE_TEMPLATE/documentation.yaml (1 hunks)
- .github/ISSUE_TEMPLATE/feature-request-application.yaml (1 hunks)
- .github/ISSUE_TEMPLATE/feature-request-website.yaml (1 hunks)
- .github/ISSUE_TEMPLATE/other.yml (1 hunks)
- .github/dependabot.yml (1 hunks)
- .github/labeler.yml (1 hunks)
- .github/linters/.jscpd.json (1 hunks)
- .github/pull_request_template.md (2 hunks)
- .github/release.yml (2 hunks)
- .github/workflows/formatter.yml (2 hunks)
- .prettierignore (1 hunks)
- CODE_OF_CONDUCT.md (2 hunks)
- CONTRIBUTING.md (1 hunks)
- Docker/macOS Docker Build Instructions.md (4 hunks)
- GUI/src/main/resources/CSS/Button.css (1 hunks)
- GUI/src/main/resources/CSS/CheckBox.css (1 hunks)
- GUI/src/main/resources/CSS/ContextMenu.css (1 hunks)
- GUI/src/main/resources/CSS/DarkTheme.css (1 hunks)
- GUI/src/main/resources/CSS/Label.css (1 hunks)
- GUI/src/main/resources/CSS/LightTheme.css (1 hunks)
- GUI/src/main/resources/CSS/ListView.css (1 hunks)
- GUI/src/main/resources/CSS/Menu.css (1 hunks)
- GUI/src/main/resources/CSS/ProgressBar.css (1 hunks)
- GUI/src/main/resources/CSS/ScrollPane.css (1 hunks)
- GUI/src/main/resources/CSS/TextField.css (1 hunks)
- GUI/src/main/resources/CSS/VBox.css (1 hunks)
- README.md (6 hunks)
- Website/app/Contribute.js (1 hunks)
- Website/app/Demo.js (1 hunks)
- Website/app/Features.js (1 hunks)
- Website/app/Footer.js (1 hunks)
- Website/app/Releases.js (7 hunks)
- Website/app/about/page.js (1 hunks)
- Website/app/api/version/dev/route.js (1 hunks)
- Website/app/api/version/latest/route.js (1 hunks)
- Website/app/docs/Contributing.md (2 hunks)
- Website/app/docs/Development/Building-Executables.md (2 hunks)
- Website/app/docs/Development/Quickstart.md (3 hunks)
- Website/app/docs/Development/macOS-docker-build-instructions.md (4 hunks)
- Website/app/docs/Faq.md (1 hunks)
- Website/app/docs/Overview.md (1 hunks)
- Website/app/docs/Quickstart.md (2 hunks)
- Website/app/docs/Usage/drifty-cli/Overview.md (2 hunks)
- Website/app/docs/Usage/drifty-cli/downloading-files.md (3 hunks)
- Website/app/docs/Usage/drifty-gui/downloading-files.md (1 hunks)
- Website/app/download/page.js (2 hunks)
- Website/app/globals.css (1 hunks)
- Website/app/layout.js (1 hunks)
- Website/app/page.js (1 hunks)
- Website/next.config.js (1 hunks)
- Website/postcss.config.js (1 hunks)
- Website/tailwind.config.js (1 hunks)
- docker-compose.yaml (3 hunks)
- version.json (1 hunks)
Files skipped from review due to trivial changes (45)
- .github/ISSUE_TEMPLATE/Bug-for-application.yaml
- .github/ISSUE_TEMPLATE/Bug-for-website.yaml
- .github/ISSUE_TEMPLATE/documentation.yaml
- .github/ISSUE_TEMPLATE/feature-request-application.yaml
- .github/ISSUE_TEMPLATE/feature-request-website.yaml
- .github/ISSUE_TEMPLATE/other.yml
- .github/dependabot.yml
- .github/labeler.yml
- .github/linters/.jscpd.json
- .github/pull_request_template.md
- .github/release.yml
- CONTRIBUTING.md
- GUI/src/main/resources/CSS/Button.css
- GUI/src/main/resources/CSS/CheckBox.css
- GUI/src/main/resources/CSS/ContextMenu.css
- GUI/src/main/resources/CSS/DarkTheme.css
- GUI/src/main/resources/CSS/Label.css
- GUI/src/main/resources/CSS/LightTheme.css
- GUI/src/main/resources/CSS/ListView.css
- GUI/src/main/resources/CSS/Menu.css
- GUI/src/main/resources/CSS/ProgressBar.css
- GUI/src/main/resources/CSS/ScrollPane.css
- GUI/src/main/resources/CSS/TextField.css
- GUI/src/main/resources/CSS/VBox.css
- Website/app/Contribute.js
- Website/app/Demo.js
- Website/app/Features.js
- Website/app/Footer.js
- Website/app/about/page.js
- Website/app/api/version/dev/route.js
- Website/app/api/version/latest/route.js
- Website/app/docs/Contributing.md
- Website/app/docs/Development/Building-Executables.md
- Website/app/docs/Development/Quickstart.md
- Website/app/docs/Overview.md
- Website/app/docs/Quickstart.md
- Website/app/docs/Usage/drifty-cli/Overview.md
- Website/app/download/page.js
- Website/app/layout.js
- Website/app/page.js
- Website/next.config.js
- Website/postcss.config.js
- Website/tailwind.config.js
- docker-compose.yaml
- version.json
Files skipped from review as they are similar to previous changes (2)
- .github/workflows/formatter.yml
- .prettierignore
Additional context used
Markdownlint
Website/app/docs/Usage/drifty-gui/downloading-files.md
19-19: null
Bare URL used(MD034, no-bare-urls)
README.md
100-100: null
Blank line inside blockquote(MD028, no-blanks-blockquote)
Biome
Website/app/globals.css
[error] 20-20: Expected a declaration item but instead found '@apply text-slate-100;'.
Expected a declaration item here.
(parse)
[error] 23-23: Expected a declaration item but instead found '@apply text-slate-100/50;'.
Expected a declaration item here.
(parse)
Website/app/Releases.js
[error] 337-337: Avoid passing content using the dangerouslySetInnerHTML prop.
Setting content using code can expose users to cross-site scripting (XSS) attacks
(lint/security/noDangerouslySetInnerHtml)
[error] 432-432: Avoid passing content using the dangerouslySetInnerHTML prop.
Setting content using code can expose users to cross-site scripting (XSS) attacks
(lint/security/noDangerouslySetInnerHtml)
[error] 519-519: Avoid passing content using the dangerouslySetInnerHTML prop.
Setting content using code can expose users to cross-site scripting (XSS) attacks
(lint/security/noDangerouslySetInnerHtml)
LanguageTool
Website/app/docs/Faq.md
[typographical] ~37-~37: Should there be a question mark at the end of this sentence?
Context: ...](https://discord.com/invite/DeT4jXPfkG). We will be happy to help you out.(MISSING_QUESTION_MARK2)
Website/app/docs/Usage/drifty-cli/downloading-files.md
[style] ~17-~17: You have already used this phrasing in nearby sentences. Consider replacing it to add variety to your writing.
Context: ...tination folder. Enter -.
if you want to download the video in the default downl...(REP_WANT_TO_VB)
[style] ~18-~18: You have already used this phrasing in nearby sentences. Consider replacing it to add variety to your writing.
Context: ...nloadsin most cases). -
L` if you want to use the last used directory. - the a...(REP_WANT_TO_VB)
[style] ~19-~19: You have already used this phrasing in nearby sentences. Consider replacing it to add variety to your writing.
Context: ...bsolute path of the directory where you want to save the video, e.g., `/home/user/Video...(REP_WANT_TO_VB)
CODE_OF_CONDUCT.md
[style] ~32-~32: Try using a synonym here to strengthen your wording.
Context: ...ind - Trolling, insulting or derogatory comments, and personal or political attacks - Pu...(COMMENT_REMARK)
Website/app/docs/Development/macOS-docker-build-instructions.md
[uncategorized] ~83-~83: Possible missing comma found.
Context: ...e I have it here, regardless of your ip address and your router will always be.1
(in...(AI_HYDRA_LEO_MISSING_COMMA)
Docker/macOS Docker Build Instructions.md
[style] ~32-~32: You have already used this phrasing in nearby sentences. Consider replacing it to add variety to your writing.
Context: ...h docker compose build gui ``` - If you are going to use the pre-built Drifty GUI Docker...(REP_GOING_TO_VB)
[uncategorized] ~83-~83: Possible missing comma found.
Context: ...e I have it here, regardless of your ip address and your router will always be.1
(in...(AI_HYDRA_LEO_MISSING_COMMA)
README.md
[style] ~98-~98: Consider a more expressive alternative.
Context: ...ning images with Docker Desktop. To do this, addsudo
before the docke...(DO_ACHIEVE)
[style] ~103-~103: You have already used this phrasing in nearby sentences. Consider replacing it to add variety to your writing.
Context: ...> - To run Drifty GUI docker image, you need to do the following 👇 > - For **Linux a...(REP_NEED_TO_VB)
[typographical] ~180-~180: Do not use a colon (:) before a series that is introduced by a preposition (‘below’). Remove the colon or add a noun or a noun phrase after the preposition.
Context: ...Run Anyway as present on the screen below: ||| |--|--| | ![Windows Defender ...(RP_COLON)
Additional comments not posted (5)
Website/app/docs/Faq.md (1)
37-37
: Consider adding a question mark.The sentence might be clearer with a question mark at the end to indicate it is a question.
- ➡️ For more queries related to the further improvement to this project, you can raise an (https://github.com/SaptarshiSarkar12/Drifty/issues/new/choose) or reach out to the maintainers on the [discord server](https://discord.com/invite/DeT4jXPfkG). We will be happy to help you out. + ➡️ For more queries related to the further improvement to this project, you can raise an (https://github.com/SaptarshiSarkar12/Drifty/issues/new/choose) or reach out to the maintainers on the [discord server](https://discord.com/invite/DeT4jXPfkG)? We will be happy to help you out.Likely invalid or redundant comment.
Tools
LanguageTool
[typographical] ~37-~37: Should there be a question mark at the end of this sentence?
Context: ...](https://discord.com/invite/DeT4jXPfkG). We will be happy to help you out.(MISSING_QUESTION_MARK2)
Website/app/docs/Usage/drifty-cli/downloading-files.md (2)
38-42
: YAML formatting improves readability.The reformatting of the YAML example enhances clarity and usability for users.
81-81
: Formatting changes enhance readability.The formatting adjustments improve the clarity and flow of the instructions.
Website/app/docs/Development/macOS-docker-build-instructions.md (1)
Line range hint
43-59
: Informative note on static IP addresses.The addition of this note provides valuable context and enhances user comprehension of networking concepts related to Docker usage.
README.md (1)
180-180
: Fix typographical issue by removing the colon.Remove the colon after "below" to correct the grammatical error.
Tools
LanguageTool
[typographical] ~180-~180: Do not use a colon (:) before a series that is introduced by a preposition (‘below’). Remove the colon or add a noun or a noun phrase after the preposition.
Context: ...Run Anyway as present on the screen below: ||| |--|--| | ![Windows Defender ...(RP_COLON)
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.
LGTM 👍
Bumps super-linter/super-linter from 6 to 7.
Release notes
Sourced from super-linter/super-linter's releases.
... (truncated)
Changelog
Sourced from super-linter/super-linter's changelog.
... (truncated)
Commits
02a1172
chore(main): release 7.0.0 (#6026)1d12a31
feat: prettier for additional languages (#6025)1080af3
build: run npm audit when installing dependencies (#5990)2664cb3
chore: fix formatting issues (#6028)2f0ac20
fix: avoid concurrent edits to the same file (#6027)9131ef7
fix!: remove undocumented secondary extension case (#5983)3d3274d
fix: remove intro text from stdout and stderr (#6020)48d5391
fix!: remove sql-lint (#5991)d9bf945
chore: remove file_array leftovers (#6018)Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting
@dependabot rebase
.Dependabot commands and options
You can trigger Dependabot actions by commenting on this PR:
@dependabot rebase
will rebase this PR@dependabot recreate
will recreate this PR, overwriting any edits that have been made to it@dependabot merge
will merge this PR after your CI passes on it@dependabot squash and merge
will squash and merge this PR after your CI passes on it@dependabot cancel merge
will cancel a previously requested merge and block automerging@dependabot reopen
will reopen this PR if it is closed@dependabot close
will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually@dependabot show <dependency name> ignore conditions
will show all of the ignore conditions of the specified dependency@dependabot ignore this major version
will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this minor version
will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this dependency
will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)Summary by CodeRabbit
.prettierignore
file to define directories that should be excluded from code formatting.Contributing.md
andQuickstart.md
documentation with better formatting and clearer instructions for users.