Skip to content

Professionalization#1

Merged
RenZan merged 3 commits intomainfrom
professionalization
Mar 3, 2026
Merged

Professionalization#1
RenZan merged 3 commits intomainfrom
professionalization

Conversation

@RenZan
Copy link
Owner

@RenZan RenZan commented Mar 3, 2026

No description provided.

GFM Developer added 3 commits March 3, 2026 00:45
- Add VERSION file as single source of truth (2.0.0)
- Update gfm to read version from VERSION file
- Fix temp file handling to use mktemp with trap cleanup
- Fix set -e issues with propagate_from_original (return 0)
- Fix set -e issues with SHORT_HASH_CACHE unbound variable
- Fix set -e issues with list-bugs-current unbound variable
- Fix set -e issues with propagate_to_cherry_picks (return 0)
- Make all test scripts executable
- Fix detector-negative.sh test

All 14 tests now passing (100%)

Security improvements:
- Use mktemp instead of predictable /tmp/ paths
- Proper cleanup via trap EXIT
- Consistent error handling with set -e
- Add i18n framework with English as default (GFM_LANG)
- Create _() translation function supporting en/fr
- Update help text and interactive mode to use translations
- Add test-i18n.sh to verify language switching
- Add .shellcheckrc configuration file

i18n features:
- GFM_LANG=en ./gfm help → English output
- GFM_LANG=fr ./gfm help → French output
- ./gfm help → English (default)

All 14 tests passing + new i18n test
CI/CD (GitHub Actions):
- ci.yml: Test on Ubuntu, run detector tests, i18n tests, shellcheck
- release.yml: Automated releases on version tags with checksums
- security.yml: CodeQL analysis + shellcheck security scans

GitHub Templates:
- Bug report template
- Feature request template
- Pull request template

Distribution:
- Homebrew formula (Formula/gfm.rb)

Documentation:
- CHANGELOG.md with Keep a Changelog format
- SECURITY.md with vulnerability reporting process

All 14 tests passing
@RenZan RenZan merged commit b33718f into main Mar 3, 2026
5 of 6 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