Skip to content

Feat: Code quality analysis - #8

Draft
Meldiron wants to merge 3 commits into
mainfrom
feat-code-ql-analysis
Draft

Feat: Code quality analysis#8
Meldiron wants to merge 3 commits into
mainfrom
feat-code-ql-analysis

Conversation

@Meldiron

Copy link
Copy Markdown
Contributor

What does this PR do?

Implements code quality analysis. This allows you to d

Implements code quality analysis into the repository. This allows you to run composer check to check for errors. It now also auto-checks every PR with composer check to create failed test if a code formatting is not correct.

Lastly, this PR updates existing code to pass quality analysis tests - to have proper code quality.

Test Plan

  • Implement new tests

Related PRs and Issues

No related issue.

Link your issue here if there is some

Have you read the Contributing Guidelines on issues?

Yes! 👏

This was referenced Oct 10, 2022

@stnguyen90stnguyen90 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

nvm. wrong PR

@Meldiron
Meldiron marked this pull request as draft August 13, 2026 11:11
Sign up for freeto 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.

2 participants

@Meldiron@stnguyen90