Skip to content

refactor(codebase): code conventions alignment - #9

Merged
Abviol merged 10 commits into
devfrom
refactor/code-conventions-alignment
Jul 4, 2026
Merged

refactor(codebase): code conventions alignment#9
Abviol merged 10 commits into
devfrom
refactor/code-conventions-alignment

Conversation

@Abviol

@Abviol Abviol commented Jul 4, 2026

Copy link
Copy Markdown
Owner

Description

The purpose of this PR is to refactor the existing codebase to apply all the Code Conventions.

Changes

Testing

  • Controlled the absence of any errors in terminal.
  • Controlled the absence of ESLint errors.
  • Navigated thoughout all the pages and interacted with all the components of the project in browser to ensure nothing was broken.

Checklist

  • Tested locally
  • No ESLint errors
  • No console errors
  • Types are correct
  • Documentation/comments updated

Abviol added 10 commits July 4, 2026 12:44
Rename files and directories following the Code Conventions specified in the ./docs/ code_conventions.md file.
Reorganize imports according to the Code Conventions defined in the ./docs/code_conventions.md file
Refactor the type's value of "not available" to "not_available" accroding to he Code Conventions defined in the `./docs/code_conventions.md` file.
As it require the Code Conventions specified in the `./docs/code_conventions.md` file
Remove all `console.log` as it require the Code Conventions specified in the `./docs/code_conventions.md` file.
- remove `export` keyword from each interface which is not used outside the fire it resides.
- extract non-UI related interfaces and move to `lib/types.ts` like BreadcrumbsItemType and CommentCardType
- remove unused params, variables, and interfaces
@vercel

vercel Bot commented Jul 4, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
scent Ready Ready Preview, Comment Jul 4, 2026 2:36pm

@Abviol
Abviol merged commit 83765eb into dev Jul 4, 2026
3 checks passed
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