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.

Sign up for GitHub

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

Jump to bottom

[Feature] Auto generation for OP docs #527

Merged
merged 7 commits into from
Jan 14, 2025
Merged

[Feature] Auto generation for OP docs #527

merged 7 commits into from
Jan 14, 2025

Conversation

HYLcool
Copy link
Collaborator

@HYLcool HYLcool commented Jan 3, 2025

  • Add a local pre-commit hook to automatically generate OP doc.
  • Operators.md and Operators_ZH.md are merged into one file Operators.md
  • Redirect the links to Operators_ZH.md to Operators.md

@HYLcool
+ add a pre-commit hook to automatically generate Operator document a…
6ece31d 
…nd tag the OP according to their implementation progress
@HYLcool HYLcool added documentation Improvements or additions to documentation enhancement New feature or request dj:ci/cd issues/PRs about CI/CD of Data-Juicer labels Jan 3, 2025
@HYLcool HYLcool requested review from BeachWang and yxdyc January 3, 2025 02:57
@HYLcool HYLcool self-assigned this Jan 3, 2025
@HYLcool HYLcool linked an issue Jan 3, 2025 that may be closed by this pull request
Simplifying Open Source Contributions Through Operator Tiering from Dev aspect #510
Closed
2 tasks
@HYLcool HYLcool marked this pull request as ready for review January 9, 2025 08:39
yxdyc
yxdyc approved these changes Jan 14, 2025
View reviewed changes
Copy link
Collaborator

@yxdyc yxdyc left a comment

Choose a reason for hiding this comment

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

LGTM. I resolved the conflicts and made some minor improvements. Plz double check them before merging into main.

@HYLcool
- remove Operators_ZH.md and redirect related link to Operators.md
b1fe473 
- invoke generic_visit before returning
@HYLcool
Merge branch 'feat/tiered_op_and_doc_opt' of github.com:alibaba/data-…
2a6951f 
…juicer into feat/tiered_op_and_doc_opt

� Conflicts:
�	README.md
@HYLcool HYLcool merged commit 50f480b into main Jan 14, 2025
3 checks passed
@HYLcool HYLcool deleted the feat/tiered_op_and_doc_opt branch January 14, 2025 09:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@yxdyc yxdyc yxdyc approved these changes

@BeachWang BeachWang Awaiting requested review from BeachWang

Assignees

@HYLcool HYLcool

Labels
dj:ci/cd issues/PRs about CI/CD of Data-Juicer documentation Improvements or additions to documentation enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Simplifying Open Source Contributions Through Operator Tiering from Dev aspect
2 participants
@HYLcool @yxdyc