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

API docs upload automated #184

Merged
merged 1 commit into from
Mar 10, 2022
Merged

API docs upload automated #184

merged 1 commit into from
Mar 10, 2022

Conversation

bparmar-splunk
Copy link
Contributor

Update:

  • While release, api docs will be generated and compressed using zip tool.
  • Using upload-artifact github-action module, it will allow us to download generated api docs zip till 90 days.

Update:
- While release, api docs will be generated and compressed using zip tool.
- Using upload-artifact github-action module, it will allow us to download api docs zip till 90 days.
@fantavlik
Copy link
Contributor

Hi @bparmar-splunk - this looks good but is it possible to test this in the test workflow so that we can verify that it works as expected? Or have you done that already?

@bparmar-splunk
Copy link
Contributor Author

Hi @bparmar-splunk - this looks good but is it possible to test this in the test workflow so that we can verify that it works as expected? Or have you done that already?

Hi @fantavlik,
Yes, prior to this PR, we have tried this in github actions test when commits are pushed. And we can see apidocs file listed in artifacts section. Please check attached snapshot for reference.
Screenshot 2022-03-10 at 09 55 43

@fantavlik
Copy link
Contributor

OK great yes I see that now.

@bparmar-splunk bparmar-splunk merged commit f05a40f into develop Mar 10, 2022
@ashah-splunk ashah-splunk mentioned this pull request Jun 17, 2022
@ashah-splunk ashah-splunk deleted the DVPL-10787 branch April 20, 2023 12:01
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