Skip to content

Additional README Changes for PR #57#61

Merged
fabianlim merged 3 commits intofoundation-model-stack:mainfrom
achew010:readme-fixes
Aug 2, 2024
Merged

Additional README Changes for PR #57#61
fabianlim merged 3 commits intofoundation-model-stack:mainfrom
achew010:readme-fixes

Conversation

@achew010
Copy link
Copy Markdown
Contributor

@achew010 achew010 commented Aug 1, 2024

Description

Additional README edits for PR #57

@achew010 achew010 requested a review from fabianlim as a code owner August 1, 2024 10:21
@achew010 achew010 changed the title edits to readme Additional README Changes for PR #57 Aug 1, 2024
@achew010 achew010 force-pushed the readme-fixes branch 2 times, most recently from c657b2e to 43cac75 Compare August 1, 2024 10:26
@fabianlim
Copy link
Copy Markdown
Contributor

@achew010 the version checks is not the correct solution, see #62

Copy link
Copy Markdown
Contributor

@fabianlim fabianlim left a comment

Choose a reason for hiding this comment

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

The version checks is not correct, can we just have the README fixes

Comment thread README.md Outdated
Comment thread README.md Outdated
achew010 and others added 3 commits August 2, 2024 02:54
Signed-off-by: 1000960000 user <aaron.chew1@ibm.com>
Co-authored-by: Yu Chin Fabian Lim <fabianlim@users.noreply.github.com>
Signed-off-by: 1000960000 user <aaron.chew1@ibm.com>
Signed-off-by: 1000960000 user <aaron.chew1@ibm.com>
@fabianlim fabianlim merged commit 0e15e8c into foundation-model-stack:main Aug 2, 2024
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