Skip to content

feat: sanitize and prepare Gokstad for open-source publishing#1

Merged
aaroncroberts merged 4 commits into
mainfrom
aaron/active
May 19, 2026
Merged

feat: sanitize and prepare Gokstad for open-source publishing#1
aaroncroberts merged 4 commits into
mainfrom
aaron/active

Conversation

@aaroncroberts
Copy link
Copy Markdown
Owner

Summary

  • Remove all CSU/Cleveland State identifiers, EMPLID 6003115, and institutional credentials
  • Replace institution-specific defaults with generic corp.example.com equivalents
  • Migrate build/publish target from GitLab to GitHub Packages (aaroncorberts)
  • Add GitHub Actions CI/CD: ci.yml (build + unit tests) + pr-check.yml (conventional commit title validation)
  • Add full NuGet metadata to all 6 .csproj files (Authors/Company/Copyright = Aaron C. Roberts)
  • Remove docfx; replace with per-module README.md files
  • Containerize integration tests via test/docker-compose.yml (LDAP, MySQL, MSSQL, Postgres, WireMock)
  • Make CredentialManager URL configurable with localhost:1138 WireMock default

Test plan

  • CI / build-and-test passes — dotnet build, format verify, unit tests excluding Integration trait
  • PR / validate passes — conventional commit title check
  • No CSU/csuohio.edu/EMPLID references remain in tracked files

- Remove all CSU institutional identifiers, credentials, EMPLID (6003115),
  and AES key material from all tracked files
- Replace GitLab CI/NuGet with GitHub Actions workflows and GitHub Packages
  publishing to aaroncorberts
- Add GitHub Actions: ci.yml (PR gate + branch CI with Integration filter),
  post-merge.yml, pr-check.yml (conventional commit validation),
  publish.yml (v* tag push to GitHub Packages)
- Add .editorconfig, global.json (SDK 8.0 pin), CODEOWNERS, dependabot.yml
- Add test/docker-compose.yml: OpenLDAP, MySQL, MSSQL, PostgreSQL, WireMock
- Mark all integration tests with [Trait("Category", "Integration")];
  CI runs unit tests only via --filter "Category!=Integration"
- Activate 8 MySQL integration tests; formally skip FTP/SFTP tests
- Add CredentialManagerTests, DbParameterTests, DbConfigurationTests
- Expand MSSQL/Oracle test files from 1 skeleton to 5 tests each
- Make CredentialManager.url configurable via AppSettings:CredentialManager:url
- Update all 12 .csproj files: Authors/Company/Copyright = Aaron C. Roberts
- Fix LICENSE copyright to Aaron C. Roberts 2026
- Remove docfx/ and docs/; write professional module READMEs for all 6 packages
- Add CONTRIBUTING.md, CODE_OF_CONDUCT.md, SECURITY.md, CHANGELOG.md
- Purge .beads/ from git tracking; add to .gitignore
Adds dotnet build/test/format/restore to the Claude Code permissions
allowlist so these common workflow commands don't require confirmation.
All bd/* and git/* entries were already present.
@aaroncroberts aaroncroberts merged commit 716e594 into main May 19, 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