Skip to content

#561 ADD Comment module #562

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
wants to merge 4 commits into
base: main
Choose a base branch
from
Draft

Conversation

schonfinkel
Copy link
Contributor

@schonfinkel schonfinkel commented Jun 24, 2025

Description

Type of change

  • Bug fix
  • New feature
  • Breaking change
  • Documentation update
  • Other (explain):

Checklist

  • I have read the contributing guidelines
  • The pull request title follows this format:
    ISSUE_NUMBER ACTION_TYPE Short description (e.g., 123 ADD Feature description)
  • I have added tests and documentation (if applicable)
  • My changes generate no new warnings/errors and do not break existing functionality

Additional context (optional)

Copy link

Plans 👍

Outputs can be viewed in the Terrateam Console here.

Terrateam Plan Output 👍

Expand for plan output details

Dir: code | No changes

Dir: code
Workspace: default
Success: 👍

Step: plan
Success: 👍


To apply all these changes, comment:

terrateam apply

Feedback?

Questions? Comments? Give feedback by commenting terrateam feedback <your msg>. Your message lands directly in our inbox.

Copy link

Plans ✖️

Outputs can be viewed in the Terrateam Console here.

Running plans FAILED. See Terrateam Plan Output.

After resolving the issue, run terrateam plan to execute the plan operation again.

Terrateam Plan Output ✖️

Expand for plan output details

Dir: code | ✖️

Dir: code
Workspace: default
Success: ✖️

Step: update-terrateam-github-token
Success: 👍


Step: run
Success: ✖️
Command: /github/workspace/bin/verify-commit-msg


Not all commit messages match the required structure
Commit messages must be of the form:

$GITHUB_ISSUE_1 $GITHUB_ISSUE_2 ... $GITHUB_ISSUE_N $ACTION $SUMMARY

OPTIONAL LONGER MESSAGE


Where:
$GITHUB_ISSUE can be of the form '#123' or 'org/repo#123'
$ACTION is ADD, REFACTOR, FIX, UPDATE, or REMOVE


For example:
#123 FIX Bug in confabulator that prevents users from making progress

The confabulator was working on a co-axial retrograde orientation when in reality
it necessary for it to be in a tri-axial prograde orientation.  This was preventing
users from verifying the semi-articulating socket pair.


Feedback?

Questions? Comments? Give feedback by commenting terrateam feedback <your msg>. Your message lands directly in our inbox.

@schonfinkel schonfinkel force-pushed the 561-ADD-bootstrap-comment-module branch from b4b7812 to bdf62bd Compare June 25, 2025 12:58
Copy link

Plans ✖️

Outputs can be viewed in the Terrateam Console here.

Running plans FAILED. See Terrateam Plan Output.

After resolving the issue, run terrateam plan to execute the plan operation again.

Terrateam Plan Output ✖️

Expand for plan output details

Dir: code | ✖️

Dir: code
Workspace: default
Success: ✖️

Step: update-terrateam-github-token
Success: 👍


Step: run
Success: ✖️
Command: /github/workspace/bin/verify-commit-msg


Not all commit messages match the required structure
Commit messages must be of the form:

$GITHUB_ISSUE_1 $GITHUB_ISSUE_2 ... $GITHUB_ISSUE_N $ACTION $SUMMARY

OPTIONAL LONGER MESSAGE


Where:
$GITHUB_ISSUE can be of the form '#123' or 'org/repo#123'
$ACTION is ADD, REFACTOR, FIX, UPDATE, or REMOVE


For example:
#123 FIX Bug in confabulator that prevents users from making progress

The confabulator was working on a co-axial retrograde orientation when in reality
it necessary for it to be in a tri-axial prograde orientation.  This was preventing
users from verifying the semi-articulating socket pair.


Feedback?

Questions? Comments? Give feedback by commenting terrateam feedback <your msg>. Your message lands directly in our inbox.

Copy link

Plans ✖️

Outputs can be viewed in the Terrateam Console here.

Running plans FAILED. See Terrateam Plan Output.

After resolving the issue, run terrateam plan to execute the plan operation again.

Terrateam Plan Output ✖️

Expand for plan output details

Dir: code | ✖️

Dir: code
Workspace: default
Success: ✖️

Step: update-terrateam-github-token
Success: 👍


Step: run
Success: ✖️
Command: /github/workspace/bin/verify-commit-msg


Not all commit messages match the required structure
Commit messages must be of the form:

$GITHUB_ISSUE_1 $GITHUB_ISSUE_2 ... $GITHUB_ISSUE_N $ACTION $SUMMARY

OPTIONAL LONGER MESSAGE


Where:
$GITHUB_ISSUE can be of the form '#123' or 'org/repo#123'
$ACTION is ADD, REFACTOR, FIX, UPDATE, or REMOVE


For example:
#123 FIX Bug in confabulator that prevents users from making progress

The confabulator was working on a co-axial retrograde orientation when in reality
it necessary for it to be in a tri-axial prograde orientation.  This was preventing
users from verifying the semi-articulating socket pair.


Feedback?

Questions? Comments? Give feedback by commenting terrateam feedback <your msg>. Your message lands directly in our inbox.

@schonfinkel schonfinkel force-pushed the 561-ADD-bootstrap-comment-module branch from 95c48d1 to 73e3e94 Compare June 25, 2025 21:21
Copy link

Plans ✖️

Outputs can be viewed in the Terrateam Console here.

Running plans FAILED. See Terrateam Plan Output.

After resolving the issue, run terrateam plan to execute the plan operation again.

Terrateam Plan Output ✖️

Expand for plan output details

Dir: code | ✖️

Dir: code
Workspace: default
Success: ✖️

Step: update-terrateam-github-token
Success: 👍


Step: run
Success: ✖️
Command: /github/workspace/bin/verify-commit-msg


Not all commit messages match the required structure
Commit messages must be of the form:

$GITHUB_ISSUE_1 $GITHUB_ISSUE_2 ... $GITHUB_ISSUE_N $ACTION $SUMMARY

OPTIONAL LONGER MESSAGE


Where:
$GITHUB_ISSUE can be of the form '#123' or 'org/repo#123'
$ACTION is ADD, REFACTOR, FIX, UPDATE, or REMOVE


For example:
#123 FIX Bug in confabulator that prevents users from making progress

The confabulator was working on a co-axial retrograde orientation when in reality
it necessary for it to be in a tri-axial prograde orientation.  This was preventing
users from verifying the semi-articulating socket pair.


Feedback?

Questions? Comments? Give feedback by commenting terrateam feedback <your msg>. Your message lands directly in our inbox.

@schonfinkel schonfinkel self-assigned this Jun 26, 2025
@schonfinkel schonfinkel moved this to In Progress in The Senate Jun 26, 2025
@schonfinkel schonfinkel force-pushed the 561-ADD-bootstrap-comment-module branch from 73e3e94 to de76d07 Compare June 27, 2025 14:50
Copy link

Plans ✖️

Outputs can be viewed in the Terrateam Console here.

Running plans FAILED. See Terrateam Plan Output.

After resolving the issue, run terrateam plan to execute the plan operation again.

Terrateam Plan Output ✖️

Expand for plan output details

Dir: code | ✖️

Dir: code
Workspace: default
Success: ✖️

Step: update-terrateam-github-token
Success: 👍


Step: run
Success: ✖️
Command: /github/workspace/bin/verify-commit-msg


Not all commit messages match the required structure
Commit messages must be of the form:

$GITHUB_ISSUE_1 $GITHUB_ISSUE_2 ... $GITHUB_ISSUE_N $ACTION $SUMMARY

OPTIONAL LONGER MESSAGE


Where:
$GITHUB_ISSUE can be of the form '#123' or 'org/repo#123'
$ACTION is ADD, REFACTOR, FIX, UPDATE, or REMOVE


For example:
#123 FIX Bug in confabulator that prevents users from making progress

The confabulator was working on a co-axial retrograde orientation when in reality
it necessary for it to be in a tri-axial prograde orientation.  This was preventing
users from verifying the semi-articulating socket pair.


Feedback?

Questions? Comments? Give feedback by commenting terrateam feedback <your msg>. Your message lands directly in our inbox.

Copy link

Plans ✖️

Outputs can be viewed in the Terrateam Console here.

Running plans FAILED. See Terrateam Plan Output.

After resolving the issue, run terrateam plan to execute the plan operation again.

Terrateam Plan Output ✖️

Expand for plan output details

Dir: code | ✖️

Dir: code
Workspace: default
Success: ✖️

Step: update-terrateam-github-token
Success: 👍


Step: run
Success: ✖️
Command: /github/workspace/bin/verify-commit-msg


Not all commit messages match the required structure
Commit messages must be of the form:

$GITHUB_ISSUE_1 $GITHUB_ISSUE_2 ... $GITHUB_ISSUE_N $ACTION $SUMMARY

OPTIONAL LONGER MESSAGE


Where:
$GITHUB_ISSUE can be of the form '#123' or 'org/repo#123'
$ACTION is ADD, REFACTOR, FIX, UPDATE, or REMOVE


For example:
#123 FIX Bug in confabulator that prevents users from making progress

The confabulator was working on a co-axial retrograde orientation when in reality
it necessary for it to be in a tri-axial prograde orientation.  This was preventing
users from verifying the semi-articulating socket pair.


Feedback?

Questions? Comments? Give feedback by commenting terrateam feedback <your msg>. Your message lands directly in our inbox.

Copy link

Plans ✖️

Outputs can be viewed in the Terrateam Console here.

Running plans FAILED. See Terrateam Plan Output.

After resolving the issue, run terrateam plan to execute the plan operation again.

Terrateam Plan Output ✖️

Expand for plan output details

Dir: code | ✖️

Dir: code
Workspace: default
Success: ✖️

Step: update-terrateam-github-token
Success: 👍


Step: run
Success: ✖️
Command: /github/workspace/bin/verify-commit-msg


Not all commit messages match the required structure
Commit messages must be of the form:

$GITHUB_ISSUE_1 $GITHUB_ISSUE_2 ... $GITHUB_ISSUE_N $ACTION $SUMMARY

OPTIONAL LONGER MESSAGE


Where:
$GITHUB_ISSUE can be of the form '#123' or 'org/repo#123'
$ACTION is ADD, REFACTOR, FIX, UPDATE, or REMOVE


For example:
#123 FIX Bug in confabulator that prevents users from making progress

The confabulator was working on a co-axial retrograde orientation when in reality
it necessary for it to be in a tri-axial prograde orientation.  This was preventing
users from verifying the semi-articulating socket pair.


Feedback?

Questions? Comments? Give feedback by commenting terrateam feedback <your msg>. Your message lands directly in our inbox.

Copy link

Plans ✖️

Outputs can be viewed in the Terrateam Console here.

Running plans FAILED. See Terrateam Plan Output.

After resolving the issue, run terrateam plan to execute the plan operation again.

Terrateam Plan Output ✖️

Expand for plan output details

Dir: code | ✖️

Dir: code
Workspace: default
Success: ✖️

Step: update-terrateam-github-token
Success: 👍


Step: run
Success: ✖️
Command: /github/workspace/bin/verify-commit-msg


Not all commit messages match the required structure
Commit messages must be of the form:

$GITHUB_ISSUE_1 $GITHUB_ISSUE_2 ... $GITHUB_ISSUE_N $ACTION $SUMMARY

OPTIONAL LONGER MESSAGE


Where:
$GITHUB_ISSUE can be of the form '#123' or 'org/repo#123'
$ACTION is ADD, REFACTOR, FIX, UPDATE, or REMOVE


For example:
#123 FIX Bug in confabulator that prevents users from making progress

The confabulator was working on a co-axial retrograde orientation when in reality
it necessary for it to be in a tri-axial prograde orientation.  This was preventing
users from verifying the semi-articulating socket pair.


Feedback?

Questions? Comments? Give feedback by commenting terrateam feedback <your msg>. Your message lands directly in our inbox.

Copy link

Plans ✖️

Outputs can be viewed in the Terrateam Console here.

Running plans FAILED. See Terrateam Plan Output.

After resolving the issue, run terrateam plan to execute the plan operation again.

Terrateam Plan Output ✖️

Expand for plan output details

Dir: code | ✖️

Dir: code
Workspace: default
Success: ✖️

Step: update-terrateam-github-token
Success: 👍


Step: run
Success: ✖️
Command: /github/workspace/bin/verify-commit-msg


Not all commit messages match the required structure
Commit messages must be of the form:

$GITHUB_ISSUE_1 $GITHUB_ISSUE_2 ... $GITHUB_ISSUE_N $ACTION $SUMMARY

OPTIONAL LONGER MESSAGE


Where:
$GITHUB_ISSUE can be of the form '#123' or 'org/repo#123'
$ACTION is ADD, REFACTOR, FIX, UPDATE, or REMOVE


For example:
#123 FIX Bug in confabulator that prevents users from making progress

The confabulator was working on a co-axial retrograde orientation when in reality
it necessary for it to be in a tri-axial prograde orientation.  This was preventing
users from verifying the semi-articulating socket pair.


Feedback?

Questions? Comments? Give feedback by commenting terrateam feedback <your msg>. Your message lands directly in our inbox.

Copy link

Plans ✖️

Outputs can be viewed in the Terrateam Console here.

Running plans FAILED. See Terrateam Plan Output.

After resolving the issue, run terrateam plan to execute the plan operation again.

Terrateam Plan Output ✖️

Expand for plan output details

Dir: code | ✖️

Dir: code
Workspace: default
Success: ✖️

Step: update-terrateam-github-token
Success: 👍


Step: run
Success: ✖️
Command: /github/workspace/bin/verify-commit-msg


Not all commit messages match the required structure
Commit messages must be of the form:

$GITHUB_ISSUE_1 $GITHUB_ISSUE_2 ... $GITHUB_ISSUE_N $ACTION $SUMMARY

OPTIONAL LONGER MESSAGE


Where:
$GITHUB_ISSUE can be of the form '#123' or 'org/repo#123'
$ACTION is ADD, REFACTOR, FIX, UPDATE, or REMOVE


For example:
#123 FIX Bug in confabulator that prevents users from making progress

The confabulator was working on a co-axial retrograde orientation when in reality
it necessary for it to be in a tri-axial prograde orientation.  This was preventing
users from verifying the semi-articulating socket pair.


Feedback?

Questions? Comments? Give feedback by commenting terrateam feedback <your msg>. Your message lands directly in our inbox.

Copy link

Plans ✖️

Outputs can be viewed in the Terrateam Console here.

Running plans FAILED. See Terrateam Plan Output.

After resolving the issue, run terrateam plan to execute the plan operation again.

Terrateam Plan Output ✖️

Expand for plan output details

Dir: code | ✖️

Dir: code
Workspace: default
Success: ✖️

Step: update-terrateam-github-token
Success: 👍


Step: run
Success: ✖️
Command: /github/workspace/bin/verify-commit-msg


Not all commit messages match the required structure
Commit messages must be of the form:

$GITHUB_ISSUE_1 $GITHUB_ISSUE_2 ... $GITHUB_ISSUE_N $ACTION $SUMMARY

OPTIONAL LONGER MESSAGE


Where:
$GITHUB_ISSUE can be of the form '#123' or 'org/repo#123'
$ACTION is ADD, REFACTOR, FIX, UPDATE, or REMOVE


For example:
#123 FIX Bug in confabulator that prevents users from making progress

The confabulator was working on a co-axial retrograde orientation when in reality
it necessary for it to be in a tri-axial prograde orientation.  This was preventing
users from verifying the semi-articulating socket pair.


Feedback?

Questions? Comments? Give feedback by commenting terrateam feedback <your msg>. Your message lands directly in our inbox.

Copy link

Plans ✖️

Outputs can be viewed in the Terrateam Console here.

Running plans FAILED. See Terrateam Plan Output.

After resolving the issue, run terrateam plan to execute the plan operation again.

Terrateam Plan Output ✖️

Expand for plan output details

Dir: code | ✖️

Dir: code
Workspace: default
Success: ✖️

Step: update-terrateam-github-token
Success: 👍


Step: run
Success: ✖️
Command: /github/workspace/bin/verify-commit-msg


Not all commit messages match the required structure
Commit messages must be of the form:

$GITHUB_ISSUE_1 $GITHUB_ISSUE_2 ... $GITHUB_ISSUE_N $ACTION $SUMMARY

OPTIONAL LONGER MESSAGE


Where:
$GITHUB_ISSUE can be of the form '#123' or 'org/repo#123'
$ACTION is ADD, REFACTOR, FIX, UPDATE, or REMOVE


For example:
#123 FIX Bug in confabulator that prevents users from making progress

The confabulator was working on a co-axial retrograde orientation when in reality
it necessary for it to be in a tri-axial prograde orientation.  This was preventing
users from verifying the semi-articulating socket pair.


Feedback?

Questions? Comments? Give feedback by commenting terrateam feedback <your msg>. Your message lands directly in our inbox.

Copy link

Plans ✖️

Outputs can be viewed in the Terrateam Console here.

Running plans FAILED. See Terrateam Plan Output.

After resolving the issue, run terrateam plan to execute the plan operation again.

Terrateam Plan Output ✖️

Expand for plan output details

Dir: code | ✖️

Dir: code
Workspace: default
Success: ✖️

Step: update-terrateam-github-token
Success: 👍


Step: run
Success: ✖️
Command: /github/workspace/bin/verify-commit-msg


Not all commit messages match the required structure
Commit messages must be of the form:

$GITHUB_ISSUE_1 $GITHUB_ISSUE_2 ... $GITHUB_ISSUE_N $ACTION $SUMMARY

OPTIONAL LONGER MESSAGE


Where:
$GITHUB_ISSUE can be of the form '#123' or 'org/repo#123'
$ACTION is ADD, REFACTOR, FIX, UPDATE, or REMOVE


For example:
#123 FIX Bug in confabulator that prevents users from making progress

The confabulator was working on a co-axial retrograde orientation when in reality
it necessary for it to be in a tri-axial prograde orientation.  This was preventing
users from verifying the semi-articulating socket pair.


Feedback?

Questions? Comments? Give feedback by commenting terrateam feedback <your msg>. Your message lands directly in our inbox.

Copy link

Plans ✖️

Outputs can be viewed in the Terrateam Console here.

Running plans FAILED. See Terrateam Plan Output.

After resolving the issue, run terrateam plan to execute the plan operation again.

Terrateam Plan Output ✖️

Expand for plan output details

Dir: code | ✖️

Dir: code
Workspace: default
Success: ✖️

Step: update-terrateam-github-token
Success: 👍


Step: run
Success: ✖️
Command: /github/workspace/bin/verify-commit-msg


Not all commit messages match the required structure
Commit messages must be of the form:

$GITHUB_ISSUE_1 $GITHUB_ISSUE_2 ... $GITHUB_ISSUE_N $ACTION $SUMMARY

OPTIONAL LONGER MESSAGE


Where:
$GITHUB_ISSUE can be of the form '#123' or 'org/repo#123'
$ACTION is ADD, REFACTOR, FIX, UPDATE, or REMOVE


For example:
#123 FIX Bug in confabulator that prevents users from making progress

The confabulator was working on a co-axial retrograde orientation when in reality
it necessary for it to be in a tri-axial prograde orientation.  This was preventing
users from verifying the semi-articulating socket pair.


Feedback?

Questions? Comments? Give feedback by commenting terrateam feedback <your msg>. Your message lands directly in our inbox.

Copy link

Plans ✖️

Outputs can be viewed in the Terrateam Console here.

Running plans FAILED. See Terrateam Plan Output.

After resolving the issue, run terrateam plan to execute the plan operation again.

Terrateam Plan Output ✖️

Expand for plan output details

Dir: code | ✖️

Dir: code
Workspace: default
Success: ✖️

Step: update-terrateam-github-token
Success: 👍


Step: run
Success: ✖️
Command: /github/workspace/bin/verify-commit-msg


Not all commit messages match the required structure
Commit messages must be of the form:

$GITHUB_ISSUE_1 $GITHUB_ISSUE_2 ... $GITHUB_ISSUE_N $ACTION $SUMMARY

OPTIONAL LONGER MESSAGE


Where:
$GITHUB_ISSUE can be of the form '#123' or 'org/repo#123'
$ACTION is ADD, REFACTOR, FIX, UPDATE, or REMOVE


For example:
#123 FIX Bug in confabulator that prevents users from making progress

The confabulator was working on a co-axial retrograde orientation when in reality
it necessary for it to be in a tri-axial prograde orientation.  This was preventing
users from verifying the semi-articulating socket pair.


Feedback?

Questions? Comments? Give feedback by commenting terrateam feedback <your msg>. Your message lands directly in our inbox.

Copy link

Plans ✖️

Outputs can be viewed in the Terrateam Console here.

Running plans FAILED. See Terrateam Plan Output.

After resolving the issue, run terrateam plan to execute the plan operation again.

Terrateam Plan Output ✖️

Expand for plan output details

Dir: code | ✖️

Dir: code
Workspace: default
Success: ✖️

Step: update-terrateam-github-token
Success: 👍


Step: run
Success: ✖️
Command: /github/workspace/bin/verify-commit-msg


Not all commit messages match the required structure
Commit messages must be of the form:

$GITHUB_ISSUE_1 $GITHUB_ISSUE_2 ... $GITHUB_ISSUE_N $ACTION $SUMMARY

OPTIONAL LONGER MESSAGE


Where:
$GITHUB_ISSUE can be of the form '#123' or 'org/repo#123'
$ACTION is ADD, REFACTOR, FIX, UPDATE, or REMOVE


For example:
#123 FIX Bug in confabulator that prevents users from making progress

The confabulator was working on a co-axial retrograde orientation when in reality
it necessary for it to be in a tri-axial prograde orientation.  This was preventing
users from verifying the semi-articulating socket pair.


Feedback?

Questions? Comments? Give feedback by commenting terrateam feedback <your msg>. Your message lands directly in our inbox.

Copy link

Plans ✖️

Outputs can be viewed in the Terrateam Console here.

Running plans FAILED. See Terrateam Plan Output.

After resolving the issue, run terrateam plan to execute the plan operation again.

Terrateam Plan Output ✖️

Expand for plan output details

Dir: code | ✖️

Dir: code
Workspace: default
Success: ✖️

Step: update-terrateam-github-token
Success: 👍


Step: run
Success: ✖️
Command: /github/workspace/bin/verify-commit-msg


Not all commit messages match the required structure
Commit messages must be of the form:

$GITHUB_ISSUE_1 $GITHUB_ISSUE_2 ... $GITHUB_ISSUE_N $ACTION $SUMMARY

OPTIONAL LONGER MESSAGE


Where:
$GITHUB_ISSUE can be of the form '#123' or 'org/repo#123'
$ACTION is ADD, REFACTOR, FIX, UPDATE, or REMOVE


For example:
#123 FIX Bug in confabulator that prevents users from making progress

The confabulator was working on a co-axial retrograde orientation when in reality
it necessary for it to be in a tri-axial prograde orientation.  This was preventing
users from verifying the semi-articulating socket pair.


Feedback?

Questions? Comments? Give feedback by commenting terrateam feedback <your msg>. Your message lands directly in our inbox.

@schonfinkel schonfinkel force-pushed the 561-ADD-bootstrap-comment-module branch from 9a3fb82 to b6a7fa9 Compare July 4, 2025 12:12
@schonfinkel schonfinkel force-pushed the 561-ADD-bootstrap-comment-module branch from 03ce792 to 6b8a3f8 Compare July 8, 2025 12:33
@schonfinkel schonfinkel force-pushed the 561-ADD-bootstrap-comment-module branch 2 times, most recently from 49f2228 to d017204 Compare July 11, 2025 18:38
@schonfinkel schonfinkel requested review from orbitz July 14, 2025 10:19
@schonfinkel schonfinkel force-pushed the 561-ADD-bootstrap-comment-module branch 2 times, most recently from f0860e3 to c75c81d Compare July 16, 2025 14:23
@schonfinkel
Copy link
Contributor Author

Almost working hausdorff-labs/terrateam#116 (comment)

@schonfinkel schonfinkel force-pushed the 561-ADD-bootstrap-comment-module branch from 6725a05 to d7221d0 Compare July 21, 2025 16:30
@schonfinkel schonfinkel force-pushed the 561-ADD-bootstrap-comment-module branch from d7221d0 to 1dcb7c6 Compare July 21, 2025 16:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: In Progress
Development

Successfully merging this pull request may close these issues.

Bootstrap a generic Comment module
2 participants