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.

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

Demo #15

Closed
wants to merge 38 commits into from
Closed

Demo #15

wants to merge 38 commits into from

Conversation

random1223
Copy link
Contributor

No description provided.

random1223 and others added 30 commits August 27, 2024 21:08
* fix_golangcilint_empty_result_error

* fix_golangcilint_empty_result_error
* fix_golangcilint_empty_result_error

* fix_golangcilint_empty_result_error

* fix_golangcilint_empty_result_error

* fix_golangcilint_empty_result_error
* fix_golangcilint_empty_result_error

* fix_golangcilint_empty_result_error

* fix_golangcilint_empty_result_error

* fix_golangcilint_empty_result_error

* fix_golangcilint_empty_result_error
Copy link

github-actions bot commented Aug 30, 2024

Codety
Issues found in this pull request based on Codety's default factory settings:

Java source code

File Issue
..s/java-pmd/module_api/src/main/java/com/MyApi.java 🔴 [Code Style] [UnnecessaryImport] Unused import 'java.util.List' view

javascript

File Issue
code-issue-examples/html/bad-html.html ⚪ [generic] [indent] Expected indentation of 4 space but found 1 space. view
⚪ [generic] [require-lang] Missing lang attribute in <html> tag. view
⚪ [Tag] [require-doctype] Missing <!DOCTYPE HTML> view
⚪ [generic] [indent] Expected indentation of 0 space but found 1 space. view
⚪ [Tag] [require-closing-tags] Missing closing tag for div. view
⚪ [generic] [indent] Expected indentation of 0 space but found 1 space. view
⚪ [AttributeValue] [no-duplicate-id] The id '1' is duplicated. view

IaC

File Issue
..amples/iac/terraform/vpc-3-layer-example/alb_sg.tf 🔴 [security] [CKV_AWS_23] Ensure every security group and rule has a description view
🔴 [security] [CKV_AWS_260] Ensure no security groups allow ingress from 0.0.0.0:0 to port 80 view

Name = "ALB Security group"
}
}

Choose a reason for hiding this comment

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

Codety
CodetyBot:

  • 🔴 [security] [CKV_AWS_23] Ensure every security group and rule has a description
  • 🔴 [security] [CKV_AWS_260] Ensure no security groups allow ingress from 0.0.0.0:0 to port 80

<div id="1"></div>
<div>abc
<HTML></HTML>

Choose a reason for hiding this comment

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

Codety
CodetyBot:

  • ⚪ [generic] [indent] Expected indentation of 4 space but found 1 space.
  • ⚪ [generic] [require-lang] Missing lang attribute in <html> tag.
  • ⚪ [Tag] [require-doctype] Missing <!DOCTYPE HTML>

<div id="1"></div>
<div id="1"></div>
<div>abc

Choose a reason for hiding this comment

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

Codety
CodetyBot:

  • ⚪ [generic] [indent] Expected indentation of 0 space but found 1 space.
  • ⚪ [Tag] [require-closing-tags] Missing closing tag for div.

<div id="1"></div>
<div id="1"></div>

Choose a reason for hiding this comment

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

Codety
CodetyBot:

  • ⚪ [generic] [indent] Expected indentation of 0 space but found 1 space.
  • ⚪ [AttributeValue] [no-duplicate-id] The id '1' is duplicated.

random1223 added a commit to random1223/pmd that referenced this pull request Sep 12, 2024
Added Codety into the tool list, Codety Scanner uses PMD as Java code analyzer. 
Demo: 
GitHub issue review example: https://github.com/codetyio/codety-scanner/actions/runs/10786005219/job/29912188828
GitHub Pull Request automation example: codetyio/codety-scanner#15 (comment)
@random1223 random1223 closed this Oct 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant