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.

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

Feature: Pull Requests Labelling System. #7003

Closed
wants to merge 5 commits into from

Conversation

offensive-vk
Copy link
Contributor

@offensive-vk offensive-vk commented Sep 5, 2024

Tip

Labels Pull Request Based on the File Changes.

  • .github/labeler.yml - To match the directory or file pattern that is modified.
  • .github/label-pr.yml - To automatically label pull requests using GITHUB_TOKEN.
  • .github/labeler.yml - Some labels are new, that doesn't exists in Roadmap Repository.

Live Action:

Ignore the Failing workflow for this pull request.

@offensive-vk offensive-vk marked this pull request as ready for review September 5, 2024 04:07
@offensive-vk
Copy link
Contributor Author

Relating to CI/CD

Hey, @kamranahmedse , Please review and the merge these changes as soon as possible. The workflows are failing because of this, so many red crosses in Actions tab lol.

Thank you.

@dansholds
Copy link
Contributor

Relating to CI/CD

Hey, @kamranahmedse , Please review and the merge these changes as soon as possible. The workflows are failing because of this, so many red crosses in Actions tab lol.

Thank you.

They're all failures because of you, on your branch though 😂

@offensive-vk
Copy link
Contributor Author

waltuh.mp4

oi @dansholds , so make them go green green 🌿

@kenshanta
Copy link
Contributor

@offensive-vk get it together m8

@offensive-vk
Copy link
Contributor Author

@kenshanta bro, what do you think about UK or London? idk

@dansholds
Copy link
Contributor

@offensive-vk @kenshanta lets try to keep it on topic!

@kenshanta
Copy link
Contributor

kenshanta commented Sep 6, 2024

@offensive-vk it won't be reviewed/merged until the deployment checks are resolved.
reference

@offensive-vk
Copy link
Contributor Author

offensive-vk commented Sep 7, 2024

😂 @kenshanta m8, It's not about the failing workflow, the maintainers (Kamran) are busy elsewhere, they just need some more time i guess, eh?

Workflow will pass after this pr has merged, do you trust me or not m8?

I have written thousands of Actions workflows, pipelines, dockerfiles. So dont you worry. ⚡

Keep contributing to Roadmap.sh

@kamranahmedse
Copy link
Owner

@offensive-vk we appreciate your contributions, but please avoid tagging people unnecessarily and using condescending tone in your comments. Keep it respectful and on-topic. This isn't the place for off-topic remarks or pushing others. Continued behavior like this will lead to a ban. Feel free to step back if you can't comply.

@offensive-vk
Copy link
Contributor Author

Hey Kamran, I’m really sorry if I came across the wrong way! I was just trying to be funny and wasn’t meaning to disrespect anyone. I’ll be more mindful going forward.

Thanks.

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.

4 participants