Skip to content

Feat/custom url rewrites#371

Merged
aymericzip merged 5 commits intomainfrom
feat/custom-url-rewrites
Jan 22, 2026
Merged

Feat/custom url rewrites#371
aymericzip merged 5 commits intomainfrom
feat/custom-url-rewrites

Conversation

@aymericzip
Copy link
Copy Markdown
Owner

Pull Request Guidelines

Thank you for contributing to the project! Please make sure your pull request follows these guidelines before submission.


Requirements

  • AI-generated code must be reviewed.
    Tips to identify AI-generated code:

    • Verbose or redundant JSDoc — make JSDoc synthetic and concise.
    • Abbreviations in variables or functions (e.g., d instead of dictionary) — avoid abbreviations.
    • Overly compact code — add line breaks and spacing to improve readability.
  • Tests and build must pass before requesting a review.

  • Commits must follow the Conventional Commit format.

  • The CONTRIBUTING.md file must be read before submitting a PR.


Codebase Conventions

  • Prefer arrow functions (() => {}) over function declarations (function () {}).
  • Prefer TypeScript type over interface.
    Use interface only for module augmentation.
  • For better testing and reusability, prefer one function per file.
  • Prefer ?? over || syntax.
  • Prefer import type over import for type imports.
  • Prefer named exports over default exports.
  • Avoid abbreviations in variable names:
    • locale instead of loc
    • map(dictionary => dictionary.key) instead of map(d => d.key)

@aymericzip aymericzip merged commit 17216fd into main Jan 22, 2026
2 of 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