Skip to content

Document returning ComputedRef for translated messages - #2

Merged
velis74 merged 7 commits into
mainfrom
readme-stored-messages
Oct 7, 2026
Merged

velis74 merged 7 commits into
mainfrom
readme-stored-messages

Conversation

@velis74

@velis74 velis74 commented Oct 7, 2026 •

Copy link
Copy Markdown
Contributor

No code change. 0.2.1.

  • The readme is split into an Application part (connecting a library, resolution order, run-time keys, locale changes, vue-i18n notes) and a Library part (declaring and reading strings, returning messages, run-time keys), with the roles of both stated at the top.
  • A library function that returns a translated message returns ComputedRef<string>. A returned string keeps the locale active at the time of the call, so a stored field error or notification does not follow a locale switch.
  • The readme states that the package depends only on Vue and works with any function shaped like t. vue-i18n is used in the examples and tests; its specifics are in a separate section.
  • The readme states that a library lists this package in peerDependencies and keeps it external.
  • package.json description matches the current API.
  • Removed the unused @/~ path aliases from vite.config.ts and tsconfig.json; the ~ alias pointed outside the repository.
  • Shorter CI comment on the changelog check.

@github-actions

github-actions Bot commented Oct 7, 2026 •

Copy link
Copy Markdown

✅ Incremental coverage — nothing to grade

No executable lines were changed between main and readme-stored-messages.

8 files not graded (1 ignored by pattern, 2 no added lines, 5 not source files)
File Reason
.github/workflows/ci.yml not a source file
changelog.md not a source file
package-lock.json not a source file
package.json not a source file
readme.md not a source file
src/translatable.spec.ts ignored by *.spec.*
tsconfig.json no added lines
vite.config.ts no added lines

Coverage reports: coverage/coverage-final.json (istanbul, 1 file) — incremental-coverage-check v3.0.0

@velis74
velis74 merged commit 0bed8bd into main Oct 7, 2026
2 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