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

tools: add WPT updater for specific subsystems #73

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

mertcanaltin
Copy link
Member

nodejs/node#54460 I'll test here

@mertcanaltin
Copy link
Member Author

Do we need to merge it? @panva

@panva
Copy link
Member

panva commented Sep 21, 2024

You can force push your changes to main here and run workflow_dispatch then, just make sure to force push main to what main is in nodejs/node after you're done testing.

@mertcanaltin
Copy link
Member Author

You can force push your changes to main here and run workflow_dispatch then, just make sure to force push main to what main is in nodejs/node after you're done testing.

I understand, thank you very much for teaching me this.

@mertcanaltin
Copy link
Member Author

I tried to forcibly push it to the master but it gave me a permission error because I didn't have permission :(

@panva
Copy link
Member

panva commented Sep 21, 2024

I tried to forcibly push it to the master but it gave me a permission error because I didn't have permission :(

Sorry, must be that you're part of the Triagers team.

@mertcanaltin
Copy link
Member Author

I tried to forcibly push it to the master but it gave me a permission error because I didn't have permission :(

Sorry, must be that you're part of the Triagers team.

I'm actually on the triage team.

@panva
Copy link
Member

panva commented Sep 21, 2024

I tried to forcibly push it to the master but it gave me a permission error because I didn't have permission :(

Sorry, must be that you're part of the Triagers team.

I'm actually on the triage team.

Exactly. A collaborator could push here.

@mertcanaltin
Copy link
Member Author

hmm I understand, I hope I can be soon ❤️ 😄

@mertcanaltin
Copy link
Member Author

@aduh95 Hello, I wonder if you could push this when you're free.

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