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

StandardJS Static Analysis Tool Integration #21

Closed
wants to merge 8 commits into from
Closed

Conversation

tinchil
Copy link

@tinchil tinchil commented Oct 23, 2024

This PR integrates StandardJS Static Analysis Tool into the project to help check and improve code quality within JavaScript/TypeScript projects.

Evidence of Installation:

  • package.json = included a new dependency "standard": "^17.1.2".
  • Screenshot of Successful Installation: screenshow included below showing a successful installation of the Standard_JS package onto the local nodebb repository.
Screenshot 2024-10-23 at 1 07 12 PM

Evidence of Execution:

  • standard_js-log.txt = indicates a series of log statements of standard_js being applied and executed and checking a sample Javascript project and identifying areas of code quality improvement which attests to the successful installation and execution of the standard js static analysis tool.

@tinchil tinchil closed this Oct 23, 2024
@tinchil
Copy link
Author

tinchil commented Oct 23, 2024

This pull request includes previous failed integration feature merging/integration attempts from P2, and hence the tests failed before even integrating the standardJs static analysis tool. Hence, I need to revert some changes before conducting the standardjs installation and creating a new pr

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