-
Notifications
You must be signed in to change notification settings - Fork 2
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
Rename transform to rewrite #369
base: master
Are you sure you want to change the base?
Rename transform to rewrite #369
Conversation
site/docs/src/pipeline.md
Outdated
The `pipeline/config.yaml` file specifies how original `EO` programs are transformed into initial `EO` programs. | ||
Transformation is necessary because some objects in original `EO` programs don't work. | ||
The `pipeline/config.yaml` file specifies how original `EO` programs are rewriteed into initial `EO` programs. | ||
Rewriteation is necessary because some objects in original `EO` programs don't work. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All these (funny) changes in this file should be removed because this file doesn't use the CLI commands terminology.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
sorry for that
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No problem. Just remove all changes to this file.
site/docs/src/pipeline.md
Outdated
The `pipeline/config.yaml` file specifies how original `EO` programs are rewriteed into initial `EO` programs. | ||
Rewriteation is necessary because some objects in original `EO` programs don't work. | ||
The `pipeline/config.yaml` file specifies how original `EO` programs are rewrote into initial `EO` programs. | ||
Rewriting is necessary because some objects in original `EO` programs don't work. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Additionally, I'd suggest you:
|
9361be1
to
cf750b6
Compare
@RoukayaZaki, please resolve the conflicts |
Solve #169
PR-Codex overview
The focus of this PR is to rename a command from
normalizer transform
tonormalizer rewrite
and update related documentation and command line options.Detailed summary
normalizer transform
command tonormalizer rewrite
normalizer rewrite
normalizer rewrite
command