-
Notifications
You must be signed in to change notification settings - Fork 247
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
Turndown message #803
Turndown message #803
Conversation
7569d55
to
58a4a23
Compare
README.md
Outdated
</p> | ||
<p style="color: inherit">We greatly appreciate the continued advocacy and support | ||
of the Service Weaver community. However, we have learned what we needed from | ||
our exploration, and we regret to announce that, <b>effective December 5, 2024, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Alternate wording suggestion:
However, we realized that it was hard for users to adopt Service Weaver directly since it required rewriting large parts of existing applications. Therefore Service Weaver did not see much direct use and therefore effective ...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated the text.
website/index.html
Outdated
@@ -27,6 +27,13 @@ <h1> | |||
<a href="docs.html">Read the Docs</a> | |||
</div> | |||
|
|||
<div class="announcement-banner"> | |||
<p>We regret to announce that, effective <b>December 5, 2024</b>, we will no longer be able to | |||
continue active development on the Service Weaver open-source framework. For more details, click |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
s/no longer be able to continue active/no longer continue/
README.md
Outdated
<p style="color: inherit">We greatly appreciate the continued advocacy and support | ||
of the Service Weaver community. However, we have learned what we needed from | ||
our exploration, and we regret to announce that, <b>effective December 5, 2024, | ||
we will transition Service Weaver into maintenance mode</b>. We recommend that users |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The sentence "We recommend that..." is a bit in the way of the overall flow. I think it can be removed, or maybe moved to the end of this paragraph.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good point. Yeah, I felt the same. However, we should let users know what would be a better approach maybe (to fork the repo). I moved that sentence to the end of the paragraph.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for the feedback
README.md
Outdated
</p> | ||
<p style="color: inherit">We greatly appreciate the continued advocacy and support | ||
of the Service Weaver community. However, we have learned what we needed from | ||
our exploration, and we regret to announce that, <b>effective December 5, 2024, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated the text.
README.md
Outdated
<p style="color: inherit">We greatly appreciate the continued advocacy and support | ||
of the Service Weaver community. However, we have learned what we needed from | ||
our exploration, and we regret to announce that, <b>effective December 5, 2024, | ||
we will transition Service Weaver into maintenance mode</b>. We recommend that users |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good point. Yeah, I felt the same. However, we should let users know what would be a better approach maybe (to fork the repo). I moved that sentence to the end of the paragraph.
No description provided.