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

Adding workflow for creating documentation issues #186

Merged

Conversation

VachaShah
Copy link
Contributor

Signed-off-by: Vacha Shah vachshah@amazon.com

Description

Adding workflow to automatically create issues in documentation-website repo when a needs-documentation label is added to a PR.

Issues Resolved

Closes #171

Check List

  • New functionality includes testing.
    • All tests pass
  • New functionality has been documented.
    • New functionality has javadoc added
  • Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@VachaShah VachaShah requested a review from a team May 19, 2022 16:28
Signed-off-by: Vacha Shah <vachshah@amazon.com>
@VachaShah VachaShah force-pushed the documentation-issue-workflow branch from f176621 to 8b77184 Compare May 19, 2022 16:53
@codecov-commenter
Copy link

codecov-commenter commented May 19, 2022

Codecov Report

Merging #186 (8b77184) into main (ab6fd08) will increase coverage by 0.06%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##               main     #186      +/-   ##
============================================
+ Coverage     71.37%   71.43%   +0.06%     
- Complexity     2937     2940       +3     
============================================
  Files           376      376              
  Lines         18769    18769              
  Branches       1445     1445              
============================================
+ Hits          13396    13408      +12     
+ Misses         4781     4772       -9     
+ Partials        592      589       -3     
Impacted Files Coverage Δ
...nsearch/performanceanalyzer/rca/RcaController.java 81.05% <0.00%> (+0.44%) ⬆️
...performanceanalyzer/collectors/StatsCollector.java 93.33% <0.00%> (+0.74%) ⬆️
...nceanalyzer/rca/samplers/MetricsDBFileSampler.java 84.09% <0.00%> (+9.09%) ⬆️
...nalyzer/rca/net/handler/PublishRequestHandler.java 85.71% <0.00%> (+12.24%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ab6fd08...8b77184. Read the comment docs.

@kkhatua kkhatua requested review from dzane17 and sgup432 May 19, 2022 18:38
@sruti1312
Copy link
Contributor

Build is failing because of this issue: opensearch-project/performance-analyzer#224. Should be safe to merge this PR.

@sruti1312 sruti1312 merged commit 097fe83 into opensearch-project:main May 20, 2022
@VachaShah VachaShah deleted the documentation-issue-workflow branch May 24, 2022 20:00
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.

Add a workflow for documenting features
5 participants