Skip to content

chore: setting up skeleton and adapting ci for go - #2

Merged
Frnd-me merged 2 commits into
developfrom
develop-gregor
Jul 21, 2026
Merged

chore: setting up skeleton and adapting ci for go#2
Frnd-me merged 2 commits into
developfrom
develop-gregor

Conversation

@GregorHolzer

@GregorHolzer GregorHolzer commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

📝 Summary

  • Create skeleton for go project and removed C++ related files and directories
  • Configured CI pipeline for go
  • Modified .md files to match the current project state
  • Using a Makefile instead of go build to enable more complex build processes later on (parser backend generation)

🛠 Quality Check

  • Technical Standards: The code adheres to the contribution guidelines.

🧪 Testing

  • Coverage: New tests have been added to cover all logic paths.
  • Verification: All tests passed locally before submission.

🚦 Type of Change

  • feat: New feature.
  • fix: Bug fix.
  • refactor: Logic change that neither fixes a bug nor adds a feature.
  • perf: Performance improvement.
  • docs: Documentation only.
  • style: Formatting, missing semi-colons, etc. (no logic change).
  • ci/build: Pipeline or build system changes.
  • BREAKING CHANGE (Requires ! in the type or footer).

@GregorHolzer
GregorHolzer requested a review from Frnd-me July 15, 2026 14:44
Comment thread .gitmodules
url = git@github.com:google/googletest.git
[submodule "spec"]
path = spec
url = git@github.com:CollaborativeStateMachines/Spec.git

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Please pin to a commit hash.

Comment thread csm/context_variable.go

type typedVariable[T any] struct {
name string
value T

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

You probably have to adapt this later.

Comment thread csm/event.go
type Event struct {
Topic string
Channel EventChannel
Data []ContextVariable

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Because event serialization may become messy here.

@Frnd-me
Frnd-me merged commit 89665f6 into develop Jul 21, 2026
1 check passed
@Frnd-me
Frnd-me deleted the develop-gregor branch August 5, 2026 11:34
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.

2 participants