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 README.md #1

Open
wants to merge 15 commits into
base: main
Choose a base branch
from
Open

Update README.md #1

wants to merge 15 commits into from

Conversation

franciscodua
Copy link
Owner

No description provided.

franciscodua
franciscodua commented Jun 14, 2021
View reviewed changes
README.md
- Identify new Static Analysis issues
* Commit and Pull Request Analysis with GitHub, BitBucket/Stash, GitLab (and also direct git repositories)
- Auto-comments on Commits and Pull Requests
Copy link
Owner Author

Choose a reason for hiding this comment

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

Suggested change
- Auto-comments on Commits and Pull Requests
- Auto-comments on Commits and Pull Requests

lolgab
lolgab reviewed Jun 15, 2021
View reviewed changes
Copy link

@lolgab lolgab left a comment

Choose a reason for hiding this comment

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

Hello WORLD

franciscodua
franciscodua commented Jun 16, 2021
View reviewed changes
README.md
# codacy-test

### Among Codacy’s features
Copy link
Owner Author

Choose a reason for hiding this comment

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

Codacy has a fix for the issue: Expected: 1; Actual: 0; Below

Suggested change
### Among Codacy’s features
### Among Codacy’s features

franciscodua
franciscodua commented Jun 16, 2021
View reviewed changes
README.md
### Among Codacy’s features
- Identify new Static Analysis issues
* Commit and Pull Request Analysis with GitHub, BitBucket/Stash, GitLab (and also direct git repositories)
Copy link
Owner Author

Choose a reason for hiding this comment

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

Codacy has a fix for the issue: Lists should be surrounded by blank lines

Suggested change
* Commit and Pull Request Analysis with GitHub, BitBucket/Stash, GitLab (and also direct git repositories)
* Commit and Pull Request Analysis with GitHub, BitBucket/Stash, GitLab (and also direct git repositories)

franciscodua
franciscodua commented Jun 16, 2021
View reviewed changes
README.md
- Identify new Static Analysis issues
* Commit and Pull Request Analysis with GitHub, BitBucket/Stash, GitLab (and also direct git repositories)
- Auto-comments on Commits and Pull Requests
Copy link
Owner Author

Choose a reason for hiding this comment

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

Codacy has a fix for the issue: Lists should be surrounded by blank lines

Suggested change
- Auto-comments on Commits and Pull Requests
- Auto-comments on Commits and Pull Requests

franciscodua
franciscodua commented Jun 16, 2021
View reviewed changes
test.js
@@ -5,3 +5,7 @@ console.log('Hello Universe');

console.log('Hello other stuff');

if(foo === "bar") {
console.log('stuff')
Copy link
Owner Author

Choose a reason for hiding this comment

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

Codacy has a fix for the issue: Strings must use doublequote.

Suggested change
console.log('stuff')
console.log("stuff")

franciscodua
franciscodua commented Jun 16, 2021
View reviewed changes
README.md

### Among Codacy’s features
- Identify new Static Analysis issues
Copy link
Owner Author

Choose a reason for hiding this comment

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

Codacy has a fix for the issue: Lists should be surrounded by blank lines

Suggested change
- Identify new Static Analysis issues
- Identify new Static Analysis issues

franciscodua
franciscodua commented Jun 16, 2021
View reviewed changes
README.md
### Among Codacy’s features
- Identify new Static Analysis issues
* Commit and Pull Request Analysis with GitHub, BitBucket/Stash, GitLab (and also direct git repositories)
Copy link
Owner Author

Choose a reason for hiding this comment

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

Codacy has a fix for the issue: Expected: dash; Actual: asterisk

Suggested change
* Commit and Pull Request Analysis with GitHub, BitBucket/Stash, GitLab (and also direct git repositories)
- Commit and Pull Request Analysis with GitHub, BitBucket/Stash, GitLab (and also direct git repositories)

franciscodua
franciscodua commented Jul 4, 2021
View reviewed changes
README.md
# codacy-test

### Among Codacy’s features
Copy link
Owner Author

Choose a reason for hiding this comment

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

Codacy has a fix for the issue: Expected: 1; Actual: 0; Below

Suggested change
### Among Codacy’s features
### Among Codacy’s features

franciscodua
franciscodua commented Jul 4, 2021
View reviewed changes
README.md
### Among Codacy’s features
- Identify new Static Analysis issues
* Commit and Pull Request Analysis with GitHub, BitBucket/Stash, GitLab (and also direct git repositories)
Copy link
Owner Author

Choose a reason for hiding this comment

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

Codacy has a fix for the issue: Lists should be surrounded by blank lines

Suggested change
* Commit and Pull Request Analysis with GitHub, BitBucket/Stash, GitLab (and also direct git repositories)
* Commit and Pull Request Analysis with GitHub, BitBucket/Stash, GitLab (and also direct git repositories)

franciscodua
franciscodua commented Jul 4, 2021
View reviewed changes
README.md
- Identify new Static Analysis issues
* Commit and Pull Request Analysis with GitHub, BitBucket/Stash, GitLab (and also direct git repositories)
- Auto-comments on Commits and Pull Requests
Copy link
Owner Author

Choose a reason for hiding this comment

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

Codacy has a fix for the issue: Lists should be surrounded by blank lines

Suggested change
- Auto-comments on Commits and Pull Requests
- Auto-comments on Commits and Pull Requests

franciscodua
franciscodua commented Jul 4, 2021
View reviewed changes
test.js
@@ -5,3 +5,7 @@ console.log('Hello Universe');

console.log('Hello other stuff');

if(foo === "bar") {
console.log('stuff')
Copy link
Owner Author

Choose a reason for hiding this comment

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

Codacy has a fix for the issue: Strings must use doublequote.

Suggested change
console.log('stuff')
console.log("stuff")

franciscodua
franciscodua commented Jul 4, 2021
View reviewed changes
README.md

### Among Codacy’s features
- Identify new Static Analysis issues
Copy link
Owner Author

Choose a reason for hiding this comment

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

Codacy has a fix for the issue: Lists should be surrounded by blank lines

Suggested change
- Identify new Static Analysis issues
- Identify new Static Analysis issues

franciscodua
franciscodua commented Jul 4, 2021
View reviewed changes
README.md
### Among Codacy’s features
- Identify new Static Analysis issues
* Commit and Pull Request Analysis with GitHub, BitBucket/Stash, GitLab (and also direct git repositories)
Copy link
Owner Author

Choose a reason for hiding this comment

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

Codacy has a fix for the issue: Expected: dash; Actual: asterisk

Suggested change
* Commit and Pull Request Analysis with GitHub, BitBucket/Stash, GitLab (and also direct git repositories)
- Commit and Pull Request Analysis with GitHub, BitBucket/Stash, GitLab (and also direct git repositories)

franciscodua
franciscodua commented Jul 5, 2021
View reviewed changes
test.js
@@ -5,3 +5,7 @@ console.log('Hello Universe');

console.log('Hello other stuff');

if(foo === "bar") {
console.log('stuff')
Copy link
Owner Author

Choose a reason for hiding this comment

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

Codacy has a fix for the issue: Strings must use doublequote.

Suggested change
console.log('stuff')
console.log("stuff")

franciscodua
franciscodua commented Jul 23, 2021
View reviewed changes
test.js
@@ -5,3 +5,7 @@ console.log('Hello Universe');

console.log('Hello other stuff');

if(foo === "bar") {
console.log('stuff')
Copy link
Owner Author

Choose a reason for hiding this comment

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

Codacy has a fix for the issue: Strings must use doublequote.

Suggested change
console.log('stuff')
console.log("stuff")

franciscodua
franciscodua commented Jul 27, 2021
View reviewed changes
test.js
@@ -5,3 +5,7 @@ console.log('Hello Universe');

console.log('Hello other stuff');

if(foo === "bar") {
console.log('stuff')
Copy link
Owner Author

Choose a reason for hiding this comment

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

Codacy has a fix for the issue: Strings must use doublequote.

Suggested change
console.log('stuff')
console.log("stuff")

franciscodua
franciscodua commented Nov 4, 2021
View reviewed changes
test.js
@@ -5,3 +5,7 @@ console.log('Hello Universe');

console.log('Hello other stuff');

if(foo === "bar") {
console.log('stuff')
Copy link
Owner Author

Choose a reason for hiding this comment

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

Codacy has a fix for the issue: Strings must use doublequote.

Suggested change
console.log('stuff')
console.log("stuff")

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lolgab lolgab lolgab left review comments

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
@franciscodua @lolgab