1b63765caa
Closes the remaining gaps in GitHub's community-standards check, adds explicit privacy and dependency documentation matching the plugin's "DSGVO-by-design" claim, and removes the stale upstream Crowdin artefact so the repo no longer suggests it ships its own translation pipeline. New community-health files: - CODE_OF_CONDUCT.md: project-specific, short and direct, single reporting path to kontakt@hellion-media.de - CONTRIBUTING.md: scope, accepted vs declined contributions, build and test instructions, EUPL-1.2 contribution terms, translation policy split between Hellion-specific (here) and upstream strings (Chat 2 repo) - SUPPORT.md: routing for bugs, security, privacy and casual feedback - .github/PULL_REQUEST_TEMPLATE.md: summary, change-type checklist, testing notes, compatibility notes for migrations and manifest fields, contribution checklist - .github/FUNDING.yml: comments-only file, no platforms enabled, points donors at the upstream Chat 2 maintainers' Ko-fi pages New privacy and compliance documentation: - PRIVACY.md: what the plugin stores locally (config, SQLite, EmoteCacheV1), retention defaults, the two outbound network calls (BetterTTV API+CDN with ShowEmotes opt-out, Square Enix Lodestone font once-off), explicit no-telemetry statement, GDPR Art. 15/17/18/20/21 rights mapped to plugin features, third-party privacy-policy links - THIRD_PARTY_NOTICES.md: direct NuGet dependencies with versions pinned to v0.5.4 (MessagePack, Microsoft.Data.Sqlite, morelinq, Pidgin, SixLabors.ImageSharp under Six Labors Split License 1.0), Dalamud SDK and .NET tooling, bundled Exo 2 font (OFL-1.1) and plugin icon, network-touch status per component, re-audit commands Crowdin cleanup: - crowdin.yml deleted (was upstream Chat 2's project_id 663694, pointed at /ChatTwo/Resources/Language.resx, never wired to HellionChat strings) - README, CONTRIBUTING and CODE_OF_CONDUCT no longer suggest HellionChat operates a Crowdin project; remaining mentions are explicitly framed as upstream Chat 2's workflow Contact and version consistency: - Maintainer email switched from maintainer@hellion-media.de to kontakt@hellion-media.de in SECURITY.md and NOTICE.md - README version references updated to 0.5.4 (header, project status block) and the update-tag pattern generalised from v0.1.x to v0.X.Y - bug_report.yml version placeholder bumped to 0.5.4 - Project-documents table added to README footer linking all health and reference files in one place Release-body automation: - .github/workflows/release.yml now extracts the matching version block from ChatTwo/HellionChat.yaml's changelog and combines it with a static install / docs footer (custom-repo URL, project document links, licence) before passing the result to softprops/action-gh-release@v3 via body_path - Workflow fails fast if no changelog block exists for the tagged version, automating the existing "yaml + repo.json + release body kept in sync" rule - Tag value passed via env: TAG_NAME with strict ^v\d+\.\d+\.\d+$ validation before any string concatenation, so the tag input cannot break out into shell evaluation
73 lines
2.3 KiB
Markdown
73 lines
2.3 KiB
Markdown
<!--
|
|
Thanks for contributing to HellionChat. Please fill in the sections
|
|
below so the review goes quickly. Delete sections that genuinely do
|
|
not apply, but do not delete the whole template.
|
|
|
|
If this is a security fix, stop here and use a private security
|
|
advisory instead:
|
|
https://github.com/JonKazama-Hellion/HellionChat/security/advisories/new
|
|
-->
|
|
|
|
## Summary
|
|
|
|
<!-- One or two sentences. What does this PR change and why. -->
|
|
|
|
## Type of change
|
|
|
|
<!-- Tick all that apply. -->
|
|
|
|
- [ ] Bug fix (non-breaking change that fixes an issue)
|
|
- [ ] New feature (non-breaking change that adds behaviour)
|
|
- [ ] Breaking change (config migration, removed feature, or behaviour
|
|
change that user-visible defaults rely on)
|
|
- [ ] Documentation only
|
|
- [ ] Translation update
|
|
- [ ] Build, CI or tooling change
|
|
- [ ] Upstream cherry-pick from Chat 2
|
|
|
|
## Linked issue
|
|
|
|
<!-- e.g. "Closes #42" or "Refs #42". For trivial typo fixes, "n/a". -->
|
|
|
|
## How I tested this
|
|
|
|
<!--
|
|
- Built locally with `dotnet build -c Release`
|
|
- Ran `dotnet test`
|
|
- Loaded the plugin in-game on Windows / Linux / macOS via XIVLauncher
|
|
- Specific scenarios I exercised in-game
|
|
-->
|
|
|
|
## User-visible changes
|
|
|
|
<!--
|
|
Anything the end user will notice. New settings, changed defaults,
|
|
new commands, new translations, removed behaviour. If none, write
|
|
"none".
|
|
-->
|
|
|
|
## Compatibility notes
|
|
|
|
<!--
|
|
- Does this require a configuration migration? If yes, which version
|
|
bump and is it covered by the existing migration tests?
|
|
- Does this change the schema in MessageStore?
|
|
- Does this change the repo.json or HellionChat.yaml manifest fields?
|
|
- Does this affect the upstream cherry-pick path? See UPSTREAM_SYNC.md.
|
|
-->
|
|
|
|
## Checklist
|
|
|
|
- [ ] I have read [CONTRIBUTING.md](../CONTRIBUTING.md) and
|
|
[CODE_OF_CONDUCT.md](../CODE_OF_CONDUCT.md).
|
|
- [ ] My change matches the existing code style (`.editorconfig`).
|
|
- [ ] I added or updated tests where the existing test infrastructure
|
|
made that practical, or I have explained why tests are not
|
|
applicable.
|
|
- [ ] I updated the README, in-plugin strings or documentation if my
|
|
change is user-visible.
|
|
- [ ] I did not include any AI-generated code without disclosing it
|
|
in the PR description (see [AI_DISCLOSURE.md](../AI_DISCLOSURE.md)).
|
|
- [ ] I confirm my contribution is released under the
|
|
[EUPL-1.2](../LICENSE).
|