-
Notifications
You must be signed in to change notification settings - Fork 362
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
chore(docs): updated the name of Twitter to X #3066
Changes from 1 commit
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Original file line number | Diff line number | Diff line change | ||||
---|---|---|---|---|---|---|
|
@@ -7,11 +7,11 @@ export interface ConstructLibraryOptions extends JsiiProjectOptions { | |||||
* | ||||||
* https://awscdk.io/packages/[@SCOPE/]PACKAGE@VERSION | ||||||
* | ||||||
* The catalog will also post a tweet to https://twitter.com/awscdkio with the | ||||||
* The catalog will also post a tweet to https://x.com/awscdkio with the | ||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Just wait a minute. I'm making the required changes. |
||||||
* package name, description and the above link. You can disable these tweets | ||||||
* through `{ announce: false }`. | ||||||
* | ||||||
* You can also add a Twitter handle through `{ twitter: 'xx' }` which will be | ||||||
* You can also add a X handle through `{ X: 'xx' }` which will be | ||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
This is an external API the can't changed unilaterally from here. |
||||||
* mentioned in the tweet. | ||||||
* | ||||||
* @see https://github.com/construct-catalog/catalog | ||||||
|
@@ -23,7 +23,7 @@ export interface ConstructLibraryOptions extends JsiiProjectOptions { | |||||
|
||||||
export interface Catalog { | ||||||
/** | ||||||
* Twitter account to @mention in announcement tweet. | ||||||
* X account to @mention in announcement tweet. | ||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Can you add something like (previously Twitter) to explain the difference in docs and property. |
||||||
*/ | ||||||
readonly twitter?: string; | ||||||
|
||||||
|
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.
Needs reverting