Skip to content

add contributing and issue templates#132

Merged
deathbyknowledge merged 7 commits into
mainfrom
contributing
Jun 25, 2026
Merged

add contributing and issue templates#132
deathbyknowledge merged 7 commits into
mainfrom
contributing

Conversation

@ka-wah

@ka-wah ka-wah commented Jun 22, 2026

Copy link
Copy Markdown
Collaborator

No description provided.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 82fdecfaa9

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

Comment thread .github/ISSUE_TEMPLATE/bug_report.yml
@ka-wah

ka-wah commented Jun 25, 2026

Copy link
Copy Markdown
Collaborator Author

@codex review please

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 6b1cd56e9a

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

Comment thread CONTRIBUTING.md Outdated
@ka-wah

ka-wah commented Jun 25, 2026

Copy link
Copy Markdown
Collaborator Author

@codex review please

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 19cb880537

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

Comment thread CONTRIBUTING.md Outdated
@ka-wah

ka-wah commented Jun 25, 2026

Copy link
Copy Markdown
Collaborator Author

@codex review

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 0bcee10213

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

Comment thread CONTRIBUTING.md Outdated
git clone https://github.com/deathbyknowledge/gsv.git
cd gsv
./scripts/setup-deps.sh # install JS deps
npm run dev # start local dev stack

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P3 Badge Don't block later setup commands behind dev

When following this setup block verbatim, this command prevents the next line from running: the root npm run dev script delegates to scripts/dev-stack.sh, which ends with exec npm exec -- wrangler dev and stays attached until interrupted. Contributors therefore won't reach the CLI build in the same terminal; move the CLI build before this step or show the dev stack as a separate-terminal command.

Useful? React with 👍 / 👎.

Comment thread CONTRIBUTING.md Outdated
For anything beyond a small fix, open an issue first so we can align before you write code.

1. Fork and branch from `main`
2. Make your changes and run tests (`npm test` / `cargo test`)

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P3 Badge Replace the missing root npm test command

The repo root package.json does not define a test script, so contributors following this instruction from the documented workflow get Missing script: "test" instead of validating JS changes. Point this at the actual workspace commands, such as gateway/web npm run test:run, or add a root test script before recommending npm test.

Useful? React with 👍 / 👎.

@deathbyknowledge deathbyknowledge merged commit 7d56698 into main Jun 25, 2026
8 checks passed
@deathbyknowledge deathbyknowledge deleted the contributing branch June 25, 2026 14:20
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