Skip to content
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.

Sign up for GitHub

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

Jump to bottom

docs: add last 02022024 office hour meeting minutes #650

Merged
merged 4 commits into from
Feb 9, 2024
Merged

docs: add last 02022024 office hour meeting minutes #650

merged 4 commits into from
Feb 9, 2024

Conversation

FaGru3n
Copy link
Contributor

@FaGru3n FaGru3n commented Feb 6, 2024 •
edited
Loading

Description

  • add meeting minutes
  • checked with npm run build / npm run serve

Preview

image

Pre-review checks

Please ensure to do as many of the following checks as possible, before asking for committer review:

tomaszbarwicki
tomaszbarwicki previously approved these changes Feb 7, 2024
View reviewed changes
Copy link
Contributor

@tomaszbarwicki tomaszbarwicki left a comment

Choose a reason for hiding this comment

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

LGTM, just a cosmetic suggestion to keep consistent capitalized wording for each bullet point across the document

@FaGru3n
Copy link
Contributor Author

FaGru3n commented Feb 7, 2024

LGTM, just a cosmetic suggestion to keep consistent capitalized wording for each bullet point across the document

adopted in 6170888

carslen
carslen previously approved these changes Feb 7, 2024
View reviewed changes
@FaGru3n FaGru3n requested a review from carslen February 9, 2024 08:38
carslen
carslen approved these changes Feb 9, 2024
View reviewed changes
Copy link
Contributor

@carslen carslen left a comment

Choose a reason for hiding this comment

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

LGTM

@FaGru3n FaGru3n merged commit f4dad72 into main Feb 9, 2024
4 checks passed
@FaGru3n FaGru3n deleted the docs/office-hour-02022024 branch February 9, 2024 08:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@SebastianBezold SebastianBezold SebastianBezold approved these changes

@carslen carslen carslen approved these changes

@almadigabor almadigabor almadigabor approved these changes

@hzierer hzierer hzierer approved these changes

@tomaszbarwicki tomaszbarwicki Awaiting requested review from tomaszbarwicki

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@FaGru3n @SebastianBezold @tomaszbarwicki @carslen @almadigabor @hzierer