Skip to content

feat: Updated from labgpt testing#319

Open
sisiranair wants to merge 8 commits intomainfrom
sisiranair/labgpt-related
Open

feat: Updated from labgpt testing#319
sisiranair wants to merge 8 commits intomainfrom
sisiranair/labgpt-related

Conversation

@sisiranair
Copy link
Copy Markdown
Contributor

Hi Katy - I have made changes addressing several feedback from labgpt output review.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 2, 2026

Hi there @sisiranair! 👋

This PR Title does not meet the Conventional Commits specification.

Add a prefix to indicate what kind of release this pull request corresponds to.
Common Types:

  • feat: A new feature | Adding a Page/Section to the Handbook
  • fix: A bug fix | Fixing a typo in the Handbook
  • style: Changes that do not affect the meaning of the code (white-space, formatting, missing semi-colons, etc)
  • perf: A code change that improves performance
  • build: Changes that affect the build system or external dependencies (example: Adding a dependency to Pixi)
  • ci: Changes to our CI configuration files and scripts (example scopes: Travis, Circle, BrowserStack, SauceLabs)
  • chore: Other changes that don't modify src or test files

Please Edit the PR Title to prepend the prefix to your PR:
[prefix]: Updated from labgpt testing

@sisiranair sisiranair changed the title Updated from labgpt testing feat: Updated from labgpt testing Apr 2, 2026
Copy link
Copy Markdown
Contributor

@strixy16 strixy16 left a comment

Choose a reason for hiding this comment

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

Looks good! I made a couple rearrangement suggestions for the info you added.

Also, just make sure when you add new pages to update the .pages files in that section so your addition is seen. Testing with pixi run serve is a quick way to double check this.

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.

Would this make more sense to have in the Onboarding section?

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.

This is a duplicate, see line 65-67

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.

This should be a sub-page under lab meeting or can go on the actual lab meeting page under the Status Updates heading

@sisiranair sisiranair requested a review from strixy16 April 7, 2026 18:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Backlog

Development

Successfully merging this pull request may close these issues.

2 participants