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

Allow workflow filenames, not just workflow IDs #39

Merged
merged 2 commits into from Oct 24, 2020
Merged

Conversation

palasha
Copy link
Contributor

@palasha palasha commented Oct 19, 2020

https://docs.github.com/en/free-pro-team@latest/rest/reference/actions#list-workflow-runs

Github supports passing the workflow filenames instead of workflow ids. Under the worklow_id parameter, it says, "You can also pass the workflow file name as a string."

Unfortunately, octokit's typescript bindings dont support this, so I need to ignore one line to allow the string through. At runtime, this works without issues.

Palash Agarwal added 2 commits October 19, 2020 12:08
Github supports passing the workflow filenames instead of workflow ids.
Unfortunately, octokit's typescript bindings dont support this, so we
need to ignore one line.

Pull Request Branch: palasha/support_filename_based_workflow_ids
Pull Request Link: #1
…kflow_ids

Support filename based workflow_ids
@styfle
Copy link
Owner

styfle commented Oct 20, 2020

Very cool! This will fix #2.

Is there an upstream PR to fix octokit types?

@palasha
Copy link
Contributor Author

palasha commented Oct 21, 2020

octokit/types.ts#155

I believe this is the issue where its being tracked, but its hard to figure out with all the different octokit repos.

@styfle styfle linked an issue Oct 21, 2020 that may be closed by this pull request
Copy link
Owner

@styfle styfle left a comment

Choose a reason for hiding this comment

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

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support for :workflow_file_name
2 participants