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

feat(GitHub Node): Add workflow resource operations #10744

Merged
merged 3 commits into from
Oct 23, 2024

Conversation

idanfishman
Copy link
Contributor

@idanfishman idanfishman commented Sep 9, 2024

Summary

Add the ability to use GitHub workflows as Nodes:

  • Disable a workflow
  • Dispatch a workflow event
  • Enable a workflow
  • Get a workflow
  • Get the usage of a workflow
  • List workflows

Following GitHub's API.

It enhances the integration between n8n and GitHub Actions Workflows, enabling greater automation by using GitHub nodes instead of HTTP nodes.

Screenshot 2024-09-09 at 20 03 31

Review / Merge checklist

@CLAassistant
Copy link

CLAassistant commented Sep 9, 2024

CLA assistant check
All committers have signed the CLA.

@idanfishman idanfishman changed the title feat(GitHub Node): Add workflow dispatch event operation feat(GitHub Node): Add workflow resource operations Sep 9, 2024
@n8n-assistant n8n-assistant bot added community Authored by a community member node/improvement New feature or request labels Sep 10, 2024
@Joffcom Joffcom removed node/improvement New feature or request community Authored by a community member labels Sep 10, 2024
@n8n-assistant n8n-assistant bot added community Authored by a community member node/improvement New feature or request in linear Issue or PR has been created in Linear for internal review labels Sep 10, 2024
@Joffcom
Copy link
Member

Joffcom commented Sep 10, 2024

Hey @idanfishman,

Thanks for the PR, We have created "GHC-221" as the internal reference to get this reviewed.

One of us will be in touch if there are any changes needed, in most cases this is normally within a couple of weeks but it depends on the current workload of the team.

2 similar comments
@Joffcom
Copy link
Member

Joffcom commented Sep 10, 2024

Hey @idanfishman,

Thanks for the PR, We have created "GHC-221" as the internal reference to get this reviewed.

One of us will be in touch if there are any changes needed, in most cases this is normally within a couple of weeks but it depends on the current workload of the team.

@Joffcom
Copy link
Member

Joffcom commented Sep 10, 2024

Hey @idanfishman,

Thanks for the PR, We have created "GHC-221" as the internal reference to get this reviewed.

One of us will be in touch if there are any changes needed, in most cases this is normally within a couple of weeks but it depends on the current workload of the team.

@idanfishman
Copy link
Contributor Author

Hey @idanfishman,

Thanks for the PR, We have created "GHC-221" as the internal reference to get this reviewed.

One of us will be in touch if there are any changes needed, in most cases this is normally within a couple of weeks but it depends on the current workload of the team.

Any progress?

@idanfishman
Copy link
Contributor Author

@Joffcom Hey,
I have fixed the import statement, I see that the tests ran successfully can we merge it already?

@Joffcom
Copy link
Member

Joffcom commented Oct 4, 2024

Hey @idanfishman,

I plan to do some PR reviews next week to get back on top of them but this will be one of the ones I plan to test and get merged.

@idanfishman
Copy link
Contributor Author

@Joffcom Anything?

@Joffcom
Copy link
Member

Joffcom commented Oct 14, 2024

Hey @idanfishman

It is booked in for this week, sadly with feature changes they get moved in favour of bug fixes.

Copy link
Member

@Joffcom Joffcom left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good

@Joffcom Joffcom merged commit d309112 into n8n-io:master Oct 23, 2024
15 checks passed
@Joffcom
Copy link
Member

Joffcom commented Oct 23, 2024

@idanfishman thanks for this, It will be available in the release this week 🙏🏻

@idanfishman
Copy link
Contributor Author

@idanfishman thanks for this, It will be available in the release this week 🙏🏻

Thanks!

@github-actions github-actions bot mentioned this pull request Oct 24, 2024
@janober
Copy link
Member

janober commented Oct 24, 2024

Got released with [email protected]

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
community Authored by a community member in linear Issue or PR has been created in Linear for internal review node/improvement New feature or request Released
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants