Skip to content

Align version strategy with Hyku; document in README#46

Merged
ShanaLMoore merged 1 commit intomainfrom
version-strategy
Mar 4, 2026
Merged

Align version strategy with Hyku; document in README#46
ShanaLMoore merged 1 commit intomainfrom
version-strategy

Conversation

@ShanaLMoore
Copy link
Copy Markdown
Contributor

  • Document version alignment: Knapsack major version matches Hyku series (6→6, 7→7)
  • Clarify Installation: clone Knapsack at desired version; submodule is already pinned to a compatible Hyku commit—no editing Hyku or submodule Gemfile
  • Reference .gitmodules for submodule pointer; remove literal branch/tag examples
  • Clean version.rb: remove test constant, keep VERSION 6.0.0

Copy link
Copy Markdown
Contributor

@orangewolf orangewolf left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks great

Comment thread README.md Outdated
Copy link
Copy Markdown
Contributor

@kirkkwang kirkkwang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think that looks good! Just left a non-blocking comment.

- Document version alignment: Knapsack major version matches Hyku series (6→6, 7→7)
- Clarify Installation: clone Knapsack at desired version; submodule is already
  pinned to a compatible Hyku commit—no editing Hyku or submodule Gemfile
- Reference .gitmodules for submodule pointer; remove literal branch/tag examples
- Clean version.rb: remove test constant, keep VERSION 6.0.0

Made-with: Cursor
@ShanaLMoore ShanaLMoore merged commit 82fcded into main Mar 4, 2026
5 checks passed
@ShanaLMoore ShanaLMoore deleted the version-strategy branch March 4, 2026 14:51
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.

3 participants