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

[CI] Setup GitHub Action to tweet when issue/tweet label is applied #359

Open
leecalcote opened this issue Dec 2, 2022 · 8 comments · May be fixed by #398
Open

[CI] Setup GitHub Action to tweet when issue/tweet label is applied #359

leecalcote opened this issue Dec 2, 2022 · 8 comments · May be fixed by #398
Assignees
@VaibhavMalik4187
Labels
area/ci Continuous integration | Build and release help wanted Extra attention is needed kind/enhancement New feature or request language/markdown Issues or pull requests that use Markdown

Comments

@leecalcote
Copy link
Member

Current Behavior

Many new and existing issues can benefit from being posted on Twitter for broader reach of potential contributors and for recognizing the efforts of existing contributors.

Desired Behavior

  1. Upon application of the issue/tweet label, tweet link to issue from smp_spec account.

Implementation

  1. Use https://github.com/marketplace/actions/send-tweet-action.
    1. status: should include the title of the [issue | pull request] - link-to-issue-or-pr #opensource
    2. e.g. "[CI] Setup GitHub Action to tweet when issue/tweet label is applied - [CI] Setup GitHub Action to tweet when issue/tweet label is applied meshery/meshery#6612 #opensource"
  2. Request or create each necessary GitHub secret at the organization level.

Acceptance Tests

  1. All service-mesh-performance repos have issue/tweet label, which has color of #eee.
  2. The GitHub Process document is updated.
  3. Label should work on both issues and pull requests.

Contributor Guides and Resources
@leecalcote leecalcote added kind/enhancement New feature or request help wanted Extra attention is needed area/ci Continuous integration | Build and release language/markdown Issues or pull requests that use Markdown labels Dec 2, 2022
@Rahilsiddique
Copy link

hey, I would like to work on this issue can I get more insights on how to go about it?

@gyohuangxin
Copy link
Member

@Rahilsiddique Great, please go ahead! I think the implementation section in the issue description is a good guide, feel free if you need more help.

@Rahilsiddique Rahilsiddique mentioned this issue Dec 10, 2022
Open
1 task
@ayushthe1
Copy link
Contributor

hey @gyohuangxin ,can i work on this issue ?

@gyohuangxin
Copy link
Member

gyohuangxin commented Dec 25, 2022 •
edited
Loading

hey @gyohuangxin ,can i work on this issue ?

@Rahilsiddique has raised a PR but it is pending to be reviewed. So @ayushthe1 can you review it and leave any comments?

@ayushthe1
Copy link
Contributor

hey @gyohuangxin ,can i work on this issue ?

@Rahilsiddique has raised a PR but it is pending to be reviewed. So @ayushthe1 can you review it and leave any comments?

sure ,i will do that . 👍

@VaibhavMalik4187
Copy link

@leecalcote I'd like to work on this issue.

@leecalcote
Copy link
Member Author

Forewarning, @VaibhavMalik4187: We hit a blocker previously and weren't able to overcome it. I don't imagine that recent changes to Twitter have made these challenges any easier. You're welcome to try. When you do, probably easiest to setup this integration with a separate test Twitter account.

@VaibhavMalik4187
Copy link

Forewarning, @VaibhavMalik4187: We hit a blocker previously and weren't able to overcome it. I don't imagine that recent changes to Twitter have made these challenges any easier. You're welcome to try. When you do, probably easiest to setup this integration with a separate test Twitter account.

Will keep that in mind, thanks for the heads-up.

VaibhavMalik4187 added a commit to VaibhavMalik4187/service-mesh-performance that referenced this issue Aug 17, 2023
@VaibhavMalik4187
Added action tweet when tweet label applied
9502713 
Used the tspascoal/get-user-teams-membership@v2 action to check if the
user is the member of the service-mesh-performance organization.

Used the ethomson/send-tweet-action@v1 action to send the when the
"issue/tweet" label is applied.

Fixes: service-mesh-performance#359

Signed-off-by: VaibhavMalik4187 <[email protected]>
@VaibhavMalik4187 VaibhavMalik4187 linked a pull request Aug 17, 2023 that will close this issue
Open
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@VaibhavMalik4187 VaibhavMalik4187

Labels
area/ci Continuous integration | Build and release help wanted Extra attention is needed kind/enhancement New feature or request language/markdown Issues or pull requests that use Markdown
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Added action tweet when tweet label applied VaibhavMalik4187/service-mesh-performance
5 participants
@leecalcote @VaibhavMalik4187 @gyohuangxin @Rahilsiddique @ayushthe1